Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add year and copyright owner to license #3227

Merged
merged 1 commit into from
Sep 19, 2022
Merged

Add year and copyright owner to license #3227

merged 1 commit into from
Sep 19, 2022

Conversation

trentdm
Copy link
Contributor

@trentdm trentdm commented Aug 31, 2022

The Apache 2 license has a section where the copyright entity and year should be included.

@trentdm trentdm marked this pull request as ready for review August 31, 2022 16:43
@trentdm trentdm requested a review from nvoxland as a code owner August 31, 2022 16:43
nvoxland
nvoxland approved these changes Sep 1, 2022
View reviewed changes
Copy link
Contributor

@nvoxland nvoxland 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 catching that. Good to see people paying attention to licenses : )

XDelphiGrl
XDelphiGrl approved these changes Sep 19, 2022
View reviewed changes
Copy link
Contributor

@XDelphiGrl XDelphiGrl left a comment

Choose a reason for hiding this comment

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

Copyright date of 2022 and owner Liquibase Inc. replaces boilerplate in the LICENSE.txt.

APPROVED

@nvoxland nvoxland merged commit 81600bf into liquibase:master Sep 19, 2022
@trentdm trentdm deleted the patch-1 branch September 23, 2022 18:41
@nvoxland nvoxland added this to the 1NEXT milestone Sep 30, 2022
@tabbyf00
Copy link

Thanks for your PR submission! We just finished reviewing and merging it into the 4.17.0 release on October 10, 2022. When you get a chance, could you please Star the Liquibase project? The star button is in the upper right corner of the screen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nvoxland nvoxland nvoxland approved these changes

@XDelphiGrl XDelphiGrl XDelphiGrl approved these changes

@suryaaki2 suryaaki2 suryaaki2 approved these changes

Assignees
No one assigned
Labels
autocandidate ignore-release-notes sprint2022-34
Projects
Liquibase Open Source
Archived in project
Milestone
v4.17.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@trentdm @tabbyf00 @nvoxland @XDelphiGrl @suryaaki2 @kataggart