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 clippy error checking in lint step #1163

Merged
merged 2 commits into from
Jan 18, 2020
Merged

Conversation

MarkMcCaskey
Copy link
Contributor

Review

  • Add a short description of the the change to the CHANGELOG.md file

@MarkMcCaskey MarkMcCaskey force-pushed the ci/clippy-error-check-in-lint branch from ece1823 to d1f0499 Compare January 18, 2020 00:05
@MarkMcCaskey
Copy link
Contributor Author

bors r+

bors bot added a commit that referenced this pull request Jan 18, 2020
1163: Add clippy error checking in lint step r=MarkMcCaskey a=MarkMcCaskey


# Review

- [ ] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Mark McCaskey <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jan 18, 2020

Build succeeded

@bors bors bot merged commit d1f0499 into master Jan 18, 2020
@bors bors bot deleted the ci/clippy-error-check-in-lint branch January 18, 2020 00:44
bors bot added a commit that referenced this pull request Feb 24, 2020
1245: Revert change of Ubuntu 18.04 to 16.04 r=MarkMcCaskey a=MarkMcCaskey

I didn't document why I changed it to 18.04 in #1163 ; making this PR to run CI and find out why.

It looks like CI is passing!  I have no idea why I updated to 18.04.  Perhaps I misread [the support table](https://wiki.ubuntu.com/Releases) and thought 16.04 no longer had standard support.

Either way, this PR seems good to merge.

# Review

- [ ] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Mark McCaskey <[email protected]>
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.

2 participants