Skip to content

Fix license header CI by adding Apache license to Makefile#145

Merged
Xuanwo merged 2 commits intomainfrom
copilot/fix-license-header-ci
Oct 18, 2025
Merged

Fix license header CI by adding Apache license to Makefile#145
Xuanwo merged 2 commits intomainfrom
copilot/fix-license-header-ci

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 18, 2025

Summary

This PR fixes the failing license header CI check by adding the required Apache Software Foundation license header to the Makefile.

Problem

The hawkeye license checker (used in CI) was failing because the Makefile was missing the Apache 2.0 license header that is present in all other source files in the repository.

Solution

Added the standard Apache 2.0 license header to the top of the Makefile, matching the format used consistently across all other files in the codebase (Cargo.toml, .yml files, .toml files, .rs files, etc.).

Verification

Verified locally by running the hawkeye license checker:

  • Before: ERROR hawkeye::subcommand: subcommand.rs:110 Found missing header files: ["/github/workspace/Makefile"]
  • After: INFO hawkeye::subcommand: subcommand.rs:116 No missing header file has been found.

Fixes the CI failure referenced in https://github.com/datafusion-contrib/datafusion-orc/actions/runs/18605628066/job/53054331467?pr=143

Original prompt

This section details on the original issue you should resolve

<issue_title>Fix license header CI</issue_title>
<issue_description>https://github.com/datafusion-contrib/datafusion-orc/actions/runs/18605628066/job/53054331467?pr=143</issue_description>

<agent_instructions>Fix CI by adding appropriate license header to missing files</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #144


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: waynexia <15380403+waynexia@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI by adding license headers to missing files Fix license header CI by adding Apache license to Makefile Oct 18, 2025
Copilot AI requested a review from waynexia October 18, 2025 16:54
@waynexia waynexia marked this pull request as ready for review October 18, 2025 17:11
@Xuanwo Xuanwo merged commit f2e6b60 into main Oct 18, 2025
28 checks passed
@Xuanwo Xuanwo deleted the copilot/fix-license-header-ci branch October 18, 2025 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix license header CI

3 participants