-
Notifications
You must be signed in to change notification settings - Fork 123
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
[WIP] Move CI infrastructure to GitHub Actions #3321
base: master
Are you sure you want to change the base?
Conversation
Also unify arches in CI
a1f0eaf
to
c57ea5f
Compare
9ea3830
to
de7d057
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3321 +/- ##
=======================================
Coverage 67.08% 67.08%
=======================================
Files 571 571
Lines 111044 111044
=======================================
Hits 74489 74489
Misses 36555 36555 ☔ View full report in Codecov by Sentry. |
✔️ 6b49704 -> Azure artifacts URL |
✔️ accacd9 -> artifacts URL |
✔️ accacd9 -> Azure artifacts URL |
✔️ 1cc77d6 -> artifacts URL |
✔️ 1cc77d6 -> Azure artifacts URL |
This reverts commit 1cc77d6.
✔️ c5144ef -> artifacts URL |
✔️ c5144ef -> Azure artifacts URL |
✔️ 5a28a75 -> artifacts URL |
c5190a0
to
5a00cab
Compare
✔️ c5190a0 -> artifacts URL |
✔️ 5a00cab -> artifacts URL |
✔️ 5a00cab -> Azure artifacts URL |
5a00cab
to
d4b2129
Compare
✔️ d4b2129 -> artifacts URL |
✔️ d4b2129 -> Azure artifacts URL |
b9fa846
to
832f5f3
Compare
|
✔️ 832f5f3 -> artifacts URL |
According to https://github.com/orgs/community/discussions/70492#discussioncomment-7362186, public repositories have unlimited free minutes, with a time limit for each job of about 6 hours, which should be plenty for our use-case.
Since the external CIs were used for building wheels on various platforms, we now build the following wheels with GitHub Actions:
x86_64
andarm64
(new, used to be manual), with Python versions3.9
to3.13
x86_64
andaarch64
, with Python versions3.9
to3.13
Supersedes #3213.
Would close #2359.
TODO list (not exhaustive):
azure-pipelines
comment bot (should probably be done after this PR is merged)nrn-modeldb-ci
(see Add option to test artifacts from GitHub Actions nrn-modeldb-ci#142)nrn-build-ci
(see Add option to test artifacts from GitHub Actions nrn-build-ci#80)wget
orcurl
without auth. To do this with a CLI tool, see here)-nightly
tag and correct version (see here for how setuptools-scm does that)get-last-azure-url
job inexternal.yml
withget-last-artifact-url
, and update the other ones to make it work with a GitHub comment