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 Smoke tests to CI and release flows #26

Merged
merged 5 commits into from
Dec 9, 2022

Conversation

rmichalak
Copy link
Contributor

Added smoke tests to ensure the binary we produce can be executed.

for MacOS ARM64, we are cross-compiling as there's currently no support for MacOS ARM64 image in GH Actions.
Due to the cross-compilation, we can't run a smoke test there.

@rmichalak rmichalak force-pushed the robertmichalak/smoke-tests-in-ci branch from 57eb50d to 4774f78 Compare December 7, 2022 14:10
@@ -6,7 +6,7 @@ on:
- "v*"

jobs:
aarch64-apple-darwin:
aarch64-apple-darwin-release:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I renamed these jobs to avoid name conflicts with the jobs defined in build.yml.

@rmichalak rmichalak changed the title Add Some tests to CI and release flows Add Smoke tests to CI and release flows Dec 7, 2022
toolchain: stable
override: true
target: x86_64-unknown-linux-musl
- uses: Swatinem/rust-cache@v1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may get warnings from GitHub that @v1 is out of date. I believe @v2 is enough to resolve this, i.e. there's nothing else you need to do except change the version.

Copy link
Contributor Author

@rmichalak rmichalak Dec 7, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not precisely the goal of this PR, but I also updated action versions to get rid of GH warnings.
I couldn't get rid of every warning because some actions do not have never versions released. Some PRs to track:
actions-rs/toolchain#220
actions-rs/cargo#223

@rmichalak rmichalak merged commit 8f5f5f4 into main Dec 9, 2022
@rmichalak rmichalak deleted the robertmichalak/smoke-tests-in-ci branch December 9, 2022 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants