Skip to content

Conversation

@jwndlng
Copy link
Member

@jwndlng jwndlng commented Aug 25, 2025

This PR removes the modified appendix from the LICENSE file to ensure full compliance with the Apache 2.0 license, which should remain in its original, unmodified form. This change aligns our repositories with standard practices for license documentation.

This PR removes the modified appendix from the LICENSE file to ensure full compliance with the Apache 2.0 license, which should remain in its original, unmodified form. This change aligns our repositories with standard practices for license documentation.
@jwndlng jwndlng requested a review from a team as a code owner August 25, 2025 07:51
Copy link
Contributor

@christoph-dfinity christoph-dfinity left a comment

Choose a reason for hiding this comment

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

Looks good in general. Just a question, approving so you can merge if the answer is no:

Shouldn't we also be adding the modified Appendix to this repo's README instead?

@jwndlng
Copy link
Member Author

jwndlng commented Aug 25, 2025

No, there is no need to add the appendix to the README.

@jwndlng jwndlng merged commit 475ba6a into master Aug 25, 2025
5 of 6 checks passed
@jwndlng
Copy link
Member Author

jwndlng commented Aug 25, 2025

Force-merging this PR as the failing check of test is not related to this change.

@jwndlng jwndlng deleted the jwndlng-patch-1 branch August 25, 2025 09:44
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.

2 participants