Skip to content

feat: Add Apache-2.0 license support#406

Merged
klmcadams merged 13 commits into
mainfrom
feat/apache-license
Apr 20, 2026
Merged

feat: Add Apache-2.0 license support#406
klmcadams merged 13 commits into
mainfrom
feat/apache-license

Conversation

@klmcadams
Copy link
Copy Markdown
Contributor

As the title says.

  • Checks if the file previously had a header with a different license, and if it does, it removes the license so the new one can be easily added to the file.
  • The default license is still MIT

@klmcadams klmcadams requested a review from a team as a code owner March 25, 2026 19:34
@klmcadams klmcadams linked an issue Mar 25, 2026 that may be closed by this pull request
2 tasks
@github-actions github-actions Bot added documentation Improvements or additions to documentation enhancement New features or code improvements labels Mar 25, 2026
Copy link
Copy Markdown
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Copy Markdown
Contributor

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @klmcadams I left some minor comments

Comment thread src/ansys/pre_commit_hooks/add_license_headers.py Outdated
Comment thread src/ansys/pre_commit_hooks/add_license_headers.py
Comment thread src/ansys/pre_commit_hooks/add_license_headers.py
Comment thread src/ansys/pre_commit_hooks/add_license_headers.py Outdated
@klmcadams klmcadams enabled auto-merge (squash) April 20, 2026 12:53
@klmcadams klmcadams merged commit ee95b36 into main Apr 20, 2026
35 checks passed
@klmcadams klmcadams deleted the feat/apache-license branch April 20, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New features or code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected behavior when moving to a new license

4 participants