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.

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

Add missing licenses and enable check in cpplint #352

Merged
merged 1 commit into from
Feb 18, 2022

Conversation

ChristopherHogan
Copy link
Collaborator

cpplint checks for license headers in every file by default, but we disabled that check back before we chose a license. Now that we have one, re-enable the check.

@ChristopherHogan ChristopherHogan added the skip-changelog PRs with this label won't be auto-filled in the release notes label Feb 18, 2022
@ChristopherHogan ChristopherHogan added this to In progress in 0.4.0-beta via automation Feb 18, 2022
@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 81.511% when pulling e9b05df on chogan/legal into f8e9f14 on master.

@ChristopherHogan ChristopherHogan merged commit 604d758 into master Feb 18, 2022
0.4.0-beta automation moved this from In progress to Done Feb 18, 2022
@ChristopherHogan ChristopherHogan deleted the chogan/legal branch February 18, 2022 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog PRs with this label won't be auto-filled in the release notes
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants