Skip to content

Remove copyright from headers #58

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

Closed
andygrove opened this issue Apr 27, 2019 · 0 comments · Fixed by #79
Closed

Remove copyright from headers #58

andygrove opened this issue Apr 27, 2019 · 0 comments · Fixed by #79

Comments

@andygrove
Copy link
Member

The file headers are very inconsistent right now. We should probably have CI fail if files don't have ASL headers so that this code is kept in compliance with ASL and could potentially be contributed to an Apache project in the future. We should certainly remove Copyright 2018 Grove Enterprises LLC from all headers.

benesch added a commit to benesch/sqlparser-rs that referenced this issue May 22, 2019
Standardize the license header, removing the copyright notice where
exists per apache#58. Also add a CI check to ensure that files without license
headers don't get merged.

Fix apache#58.
benesch added a commit to benesch/sqlparser-rs that referenced this issue May 22, 2019
Standardize the license header, removing the Grove Enterprise copyright
notice where it exists per apache#58. Also add a CI check to ensure that files
without license headers don't get merged.

Fix apache#58.
benesch added a commit to benesch/sqlparser-rs that referenced this issue May 22, 2019
Standardize the license header, removing the Grove Enterprise copyright
notice where it exists per apache#58. Also add a CI check to ensure that files
without license headers don't get merged.

Fix apache#58.
benesch added a commit to benesch/sqlparser-rs that referenced this issue May 22, 2019
Standardize the license header, removing the Grove Enterprise copyright
notice where it exists per apache#58. Also add a CI check to ensure that files
without license headers don't get merged.

Fix apache#58.
benesch added a commit to benesch/sqlparser-rs that referenced this issue Jun 4, 2019
Standardize the license header, removing the Grove Enterprise copyright
notice where it exists per apache#58. Also add a CI check to ensure that files
without license headers don't get merged.

Fix apache#58.
benesch added a commit to benesch/sqlparser-rs that referenced this issue Jun 4, 2019
Standardize the license header, removing the Grove Enterprise copyright
notice where it exists per apache#58. Also add a CI check to ensure that files
without license headers don't get merged.

Fix apache#58.
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 a pull request may close this issue.

1 participant