Skip to content

Exclude c-m-rt from cron test of cortex-m, fixes #417 #418

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 3, 2022
Merged

Exclude c-m-rt from cron test of cortex-m, fixes #417 #418

merged 1 commit into from
Feb 3, 2022

Conversation

adamgreig
Copy link
Member

@adamgreig adamgreig commented Jan 29, 2022 •
edited
Loading

Excludes cortex-m-rt from the plain ci-linux cron CI. We already test cortex-m-rt separate in rt-ci-linux, and plain cargo test doesn't work on x86 for cortex-m-rt.

Fixes #417 which is where this week's cron run failed.

@adamgreig adamgreig requested a review from a team as a code owner January 29, 2022 17:56
@adamgreig
Copy link
Member Author

bors r=thejpster

@bors
Copy link
Contributor

bors bot commented Feb 3, 2022

Build succeeded:

@bors bors bot merged commit 70b4d4c into master Feb 3, 2022
@bors bors bot deleted the fix-417 branch February 3, 2022 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thejpster thejpster thejpster approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI Failure
2 participants
@adamgreig @thejpster