Skip to content

Add Smoke tests to CI and release flows#26

Merged
rmichalak merged 5 commits intomainfrom
robertmichalak/smoke-tests-in-ci
Dec 9, 2022
Merged

Add Smoke tests to CI and release flows#26
rmichalak merged 5 commits intomainfrom
robertmichalak/smoke-tests-in-ci

Conversation

@rmichalak
Copy link
Copy Markdown

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 requested review from allenap and jelder December 7, 2022 14:10
@rmichalak rmichalak force-pushed the robertmichalak/smoke-tests-in-ci branch from 57eb50d to 4774f78 Compare December 7, 2022 14:10

jobs:
aarch64-apple-darwin:
aarch64-apple-darwin-release:
Copy link
Copy Markdown
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
Comment thread .github/workflows/build.yml Outdated
toolchain: stable
override: true
target: x86_64-unknown-linux-musl
- uses: Swatinem/rust-cache@v1
Copy link
Copy Markdown
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
Copy Markdown
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.

3 participants