Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Run integration tests on all PRs #443

Closed
1 task
dhruvkb opened this issue Jan 3, 2022 · 0 comments · Fixed by #507
Closed
1 task

Run integration tests on all PRs #443

dhruvkb opened this issue Jan 3, 2022 · 0 comments · Fixed by #507
Assignees
@AetherUnbound
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon

Comments

@dhruvkb
Copy link
Member

dhruvkb commented Jan 3, 2022 •
edited
Loading

Description

Currently the integration tests only run on PRs that target main. This means PRs targeting another branch don't get tested until that branch gets merged and the base is automatically changed to main.

Expectation

Tests should run on all PRs so that a all PRs in a chain have a good amount of confidence that they don't break anything.

Additional context

To prevent merge conflicts in many open PRs, this is blocked till #438 is merged.

Resolution

  • 🙋 I would be interested in resolving this bug.
@dhruvkb dhruvkb added good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon 🚧 status: blocked 🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users labels Jan 3, 2022
@sarayourfriend sarayourfriend mentioned this issue Feb 6, 2022
Closed
@AetherUnbound AetherUnbound self-assigned this Feb 6, 2022
@AetherUnbound AetherUnbound mentioned this issue Feb 6, 2022
Merged
7 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@AetherUnbound AetherUnbound

Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Run CI/CD on every pull request WordPress/openverse-api
2 participants
@AetherUnbound @dhruvkb