-
Notifications
You must be signed in to change notification settings - Fork 61
Azure Pipelines configuration #346
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
Conversation
Not crate-ci's default of 1.32.0 https://github.com/crate-ci/azure-pipelines#minimum-supported-rust-version-msrv
…top-level annotations here
…not use top-level annotations here" This reverts commit 389df39.
…ore. Use: clippy::all
https://github.com/crate-ci/azure-pipelines looks to be the easiest way to setup a Rust project with Azure Pipelines. But it enables Clippy by default, with seemingly no option to turn it off. Forked to https://github.com/iceiix/azure-pipelines/tree/clippy to allow configuring clippy, but still haven't got it passing yet. It would be a good idea to fix all the clippy warnings, nonetheless. Many are legitimate errors. But the majority are in steven_gl, gl-generator-generated code. I want to remove this dependency with #262 glow, but that migration is not complete enough to merge. Fixing the warnings shouldn't be a blocker for configuring CI. |
and many of the form:
|
Now gets past clippy, but cargo test fails:
https://stackoverflow.com/questions/55758892/missing-libraries-on-linux-with-rust-and-amethyst suggests installing missing libraries. script task from https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/?view=azure-devops |
Azure Pipelines builds completed successfully finally. https://dev.azure.com/iceix/ice_ix/_build/results?buildId=29&view=results a pleasant surprise, Windows completed in 18m 19s, faster than AppVeyor https://ci.appveyor.com/project/iceiix/stevenarella/history 24 min 23 sec, 0.75x as long. Only pending build before this PR gets the green check is AppVeyor, still waiting (haven't removed yet). Currently tests on:
|
Travis also supports mac. |
I had Travis-CI configured previously, but was hitting build cache timeout problems I couldn't solve so I disabled it (#173). This was not long after their ownership changed (https://twitter.com/carmatrocity/status/1098538649908666368), which may or may not be related, but it appeared Travis was becoming less reliable, maybe it is better now... Also looked at CircleCI (#174) as another alternative but it aborted the build due to memory consumption, though that should be mitigated now by the fix in #267. |
Setup https://github.com/marketplace/azure-pipelines. The main advantage over our current AppVeyor (Windows) and SourceHut (Linux) CI systems is Azure also supports macOS: