-
Notifications
You must be signed in to change notification settings - Fork 207
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
ci: test release workflow #1982
Conversation
permissions: | ||
contents: write | ||
pull-requests: write | ||
env: | ||
GITHUB_TOKEN: ${{secrets.GITHUBTOKEN}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need these permissions and env?
I don't think we doing anything here except for testing that build is successful
strategy: | ||
fail-fast: false | ||
matrix: | ||
build: | ||
[ | ||
linux-x64-gnu, | ||
linux-x64-musl, | ||
linux-arm64-gnu, | ||
linux-arm64-musl, | ||
linux-ia32-gnu, | ||
darwin-arm64, | ||
darwin-x64, | ||
win32-x64-msvc, | ||
win32-x64-gnu, | ||
win32-arm64-msvc, | ||
win32-ia32-gnu, | ||
] | ||
include: | ||
- build: linux-x64-gnu | ||
os: ubuntu-latest | ||
rust: stable | ||
target: x86_64-unknown-linux-gnu | ||
libc: glibc | ||
cross: false | ||
|
||
- build: linux-x64-musl | ||
os: ubuntu-latest | ||
rust: stable | ||
target: x86_64-unknown-linux-musl | ||
libc: musl | ||
cross: true | ||
|
||
- build: linux-arm64-gnu | ||
os: ubuntu-latest | ||
rust: stable | ||
target: aarch64-unknown-linux-gnu | ||
libc: glibc | ||
cross: true | ||
|
||
- build: linux-arm64-musl | ||
os: ubuntu-latest | ||
rust: stable | ||
target: aarch64-unknown-linux-musl | ||
libc: musl | ||
cross: true | ||
|
||
- build: linux-ia32-gnu | ||
os: ubuntu-latest | ||
rust: stable | ||
target: i686-unknown-linux-gnu | ||
libc: glibc | ||
cross: true | ||
|
||
- build: darwin-arm64 | ||
os: macos-latest | ||
rust: stable | ||
target: aarch64-apple-darwin | ||
|
||
- build: darwin-x64 | ||
os: macos-latest | ||
rust: stable | ||
target: x86_64-apple-darwin | ||
cross: false | ||
|
||
- build: win32-x64-msvc | ||
os: windows-latest | ||
rust: stable | ||
target: x86_64-pc-windows-msvc | ||
libc: msvc | ||
ext: ".exe" | ||
|
||
- build: win32-x64-gnu | ||
os: windows-latest | ||
rust: stable | ||
target: x86_64-pc-windows-gnu | ||
libc: glibc | ||
ext: ".exe" | ||
|
||
- build: win32-arm64-msvc | ||
os: windows-latest | ||
rust: stable | ||
target: aarch64-pc-windows-msvc | ||
features: --no-default-features --features cli | ||
libc: msvc | ||
ext: ".exe" | ||
|
||
- build: win32-ia32-gnu | ||
os: ubuntu-latest | ||
rust: stable | ||
target: i686-pc-windows-gnu | ||
features: --no-default-features --features cli | ||
cross: true | ||
libc: glibc | ||
ext: ".exe" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it'll be hard to maintain this list in multiple places.
We should consider reusing strategy matrix. Probably with the help of https://github.com/druzsan/setup-matrix and https://docs.github.com/en/actions/using-workflows/reusing-workflows
Action required: PR inactive for 2 days. |
Moving to draft, until the comments are resolved/answered. |
Action required: PR inactive for 2 days. |
PR closed after 5 days of inactivity. |
Summary:
This PR adds a workflow for testing release build. This workflow will only be triggered if
ci: test release
label is added to a PR.Issue Reference(s):
Fixes #... (Replace "..." with the issue number)
Build & Testing:
cargo test
successfully../lint.sh --mode=fix
to fix all linting issues raised by./lint.sh --mode=check
.Checklist:
<type>(<optional scope>): <title>