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

Extensive platform-specific tests no longer run in PRs #356

Merged
merged 3 commits into from
Feb 14, 2024
Merged

Conversation

lhstrh
Copy link
Member

@lhstrh lhstrh commented Feb 14, 2024

...unless necessitated by changes to specific files. We still run all platforms in main.

@lhstrh lhstrh changed the title Disable platform-specific tests unless in master or specific files we… Prevent extensive platform-specific tests from running in PRs Feb 14, 2024
@lhstrh lhstrh added the ci label Feb 14, 2024
@lhstrh lhstrh changed the title Prevent extensive platform-specific tests from running in PRs Extensive platform-specific tests no longre run in PRs Feb 14, 2024
@lhstrh lhstrh changed the title Extensive platform-specific tests no longre run in PRs Extensive platform-specific tests no longer run in PRs Feb 14, 2024
@lhstrh lhstrh force-pushed the platform-testing branch 2 times, most recently from cb9fbea to 83c40b7 Compare February 14, 2024 07:59
@lhstrh lhstrh force-pushed the platform-testing branch 3 times, most recently from 788cd0a to 5e59cd7 Compare February 14, 2024 08:56
Copy link
Contributor

@edwardalee edwardalee left a comment

Choose a reason for hiding this comment

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

It seems that check-diff.sh is missing from this PR. I can't really evaluate it without that.

core/platform/lf_zephyr_support.c Show resolved Hide resolved
@lhstrh lhstrh merged commit 8003b4b into main Feb 14, 2024
20 checks passed
@lhstrh lhstrh deleted the platform-testing branch February 14, 2024 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants