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

ci: Disable coverage tests in PRs that do not touch code #560

Open
cason opened this issue Nov 14, 2024 · 1 comment
Open

ci: Disable coverage tests in PRs that do not touch code #560

cason opened this issue Nov 14, 2024 · 1 comment
Labels
ci Related to CI good first issue Good for newcomers tech-debt Not high priority, to eventually be addressed.

Comments

@cason
Copy link
Contributor

cason commented Nov 14, 2024

Namely,

  1. codecov/project (Codecov Report, which in particular requires the author to be "an activated member of this organization on Codecov".
  2. Coverage / Integration (pull_request), which also checks Rust code

This request is particular relevant for spec and documentation PRs, where we don't touch any Rust code.

@cason cason added test Testing tech-debt Not high priority, to eventually be addressed. and removed test Testing labels Nov 14, 2024
@cason
Copy link
Contributor Author

cason commented Nov 14, 2024

Note that PRs can still be merged when those tests fail or are incomplete, so this is not really blocking anyone.

@romac romac changed the title ci: disable coverage tests in PRs that do not touch code ci: Disable coverage tests in PRs that do not touch code Dec 19, 2024
@romac romac added good first issue Good for newcomers ci Related to CI labels Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Related to CI good first issue Good for newcomers tech-debt Not high priority, to eventually be addressed.
Projects
None yet
Development

No branches or pull requests

2 participants