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

Fixes to CI. #80

Merged
merged 2 commits into from
Feb 10, 2020
Merged

Fixes to CI. #80

merged 2 commits into from
Feb 10, 2020

Conversation

mjp41
Copy link
Member

@mjp41 mjp41 commented Feb 10, 2020

Fixing some bugs in CI.

  • Make checks for copyright/license in files
  • Move to clang-format-9.

@mjp41 mjp41 requested a review from plietar February 10, 2020 12:04
@mjp41 mjp41 force-pushed the fixups branch 2 times, most recently from 92e787b to 17f8650 Compare February 10, 2020 14:05
The default behaviour for clangfprmat-9 has AfterCaseLabel: false,
but the code has used it implicitly as true.
This means that we have a breaking change from 8 to 9.
This commit moves us to 9.
@mjp41 mjp41 changed the title Fixs to CI. Fixes to CI. Feb 10, 2020
@plietar plietar merged commit 0c90d94 into master Feb 10, 2020
@plietar plietar deleted the fixups branch February 10, 2020 16:09
@mjp41 mjp41 mentioned this pull request Feb 10, 2020
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.

None yet

2 participants