Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add new and updated licenses#3586

Merged
AyanSinhaMahapatra merged 4 commits intodevelopfrom
license-sync
Nov 16, 2023
Merged

Add new and updated licenses#3586
AyanSinhaMahapatra merged 4 commits intodevelopfrom
license-sync

Conversation

Copy link
Member

AyanSinhaMahapatra commented Nov 16, 2023 *
edited
Loading

Thanks for the new licenses and updates @DennisClark

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled and links the original issue above
  • Tests pass -- look for a green checkbox a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts
  • Looked for possible updates in documentation and added updates if applicable
  • Updated CHANGELOG.rst

These were added by @DennisClark

Signed-off-by: Ayan Sinha Mahapatra
AyanSinhaMahapatra force-pushed the license-sync branch from eda9b4c to 875ec6b Compare November 16, 2023 15:14
AyanSinhaMahapatra requested review from DennisClark and pombredanne November 16, 2023 16:18
Reference: #3571

Signed-off-by: Ayan Sinha Mahapatra
DennisClark reviewed Nov 16, 2023
DennisClark requested changes Nov 16, 2023
Copy link
Member

DennisClark left a comment

Choose a reason for hiding this comment

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

@AyanSinhaMahapatra everything looks good, except please do not change the owner of niels-ferguson "Niels Ferguson License" to Unspecified but leave it as "Niels Ferguson". Thanks.

AyanSinhaMahapatra reacted with thumbs up emoji
Signed-off-by: Ayan Sinha Mahapatra
DennisClark approved these changes Nov 16, 2023
Copy link
Member

DennisClark left a comment

Choose a reason for hiding this comment

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

@AyanSinhaMahapatra Looks ready to deploy, thanks.

AyanSinhaMahapatra reacted with rocket emoji
Signed-off-by: Ayan Sinha Mahapatra
AyanSinhaMahapatra merged commit 8de1e90 into develop Nov 16, 2023
AyanSinhaMahapatra deleted the license-sync branch November 16, 2023 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

DennisClark DennisClark approved these changes

pombredanne Awaiting requested review from pombredanne

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants