Skip to content

In binary distributions, now we group the artifacts and we add BSD/MIT licenses inline #1573

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 16, 2025

Conversation

jbonofre
Copy link
Member

My script to help on LICENSE didn't include the required license content inline (for BSD or MIT licenses).

I updated my script:

  1. To group the artifacts (to reduce the duplicate content)
  2. Add BSD and MIT licenses inline

@jbonofre
Copy link
Member Author

I'm doing a new complete pass to double check all is there. @justinmclean do you mind to take a quick look ? Thanks !

flyrain
flyrain previously approved these changes May 12, 2025
Copy link
Contributor

@flyrain flyrain left a comment

Choose a reason for hiding this comment

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

It makes sense to group them. Thanks @jbonofre !

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board May 12, 2025
dimas-b
dimas-b previously approved these changes May 13, 2025
@jbonofre jbonofre dismissed stale reviews from dimas-b and flyrain via 004f517 May 13, 2025 05:27
@jbonofre jbonofre force-pushed the license-inline branch 3 times, most recently from 22c321d to bd2dba0 Compare May 13, 2025 14:43
Group: software.amazon.awssdk Name: sts Version: 2.31.30
Group: software.amazon.awssdk Name: third-party-jackson-core Version: 2.31.30
Group: software.amazon.awssdk Name: utils Version: 2.31.30
Group: software.amazon.awssdk Name: apache-client Version: 2.31.40
Copy link
Member

Choose a reason for hiding this comment

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

The version here changed, does that mean we missed an AWS bump in the notice?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I ran my script again and I found that several versions in LICENSE and NOTICE were not up to date.

@jbonofre jbonofre added this to the 0.10.0-beta milestone May 14, 2025
@jbonofre jbonofre merged commit e4287a8 into apache:main May 16, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board May 16, 2025
@jbonofre jbonofre deleted the license-inline branch May 16, 2025 02:53
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.

5 participants