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

GitHub Actions: run clippy, check formatting #365

Merged
merged 16 commits into from
Jul 5, 2020
Merged

GitHub Actions: run clippy, check formatting #365

merged 16 commits into from
Jul 5, 2020

Conversation

iceiix
Copy link
Owner

@iceiix iceiix commented Jul 5, 2020

part of #352 CI/CD consolidation/enhancements

@iceiix
Copy link
Owner Author

iceiix commented Jul 5, 2020

Not sure why this change is failing… first two (on Windows and Ubuntu) were cancelled for some reason, and macOS failed, but with not much information on why:

Screen Shot 2020-07-04 at 6 08 36 PM

@iceiix
Copy link
Owner Author

iceiix commented Jul 5, 2020

https://github.saobby.my.eu.orgmunity/t/build-got-was-canceled-unexpectedly/16130/7 explains the first two builds are canceled because the last failed, "fail fast". The clippy task is failing. The example at https://github.com/actions-rs/clippy-check#example-workflow adds components: clippy to the actions-rs/toolchain task.

@iceiix iceiix merged commit e297342 into master Jul 5, 2020
iceiix added a commit that referenced this pull request Jul 5, 2020
Consolidates the multitude of CI systems, replacing Azure, SourceHut, and
AppVeyor with GitHub Actions, which now also runs clippy and fmt. See #352

* Add task to run clippy

* Add task to check formatting

* Test on 1.44.1 toolchain

* Update readme: status badge and downloads on GitHub Actions

* Remove strip binary task, leave it unstripped to allow debugging for now

* Rename task to install dependencies

* Remove Azure Pipelines

* Remove sr.ht SourceHut CI

* Remove AppVeyor
@iceiix iceiix deleted the ghactions2 branch August 9, 2020 23:24
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.

1 participant