Skip to content

ci: add pull request trigger to CI workflow#528

Merged
dpascualhe merged 1 commit intoJdeRobot:masterfrom
Armaan-Raisinghani:fix/ci
Apr 17, 2026
Merged

ci: add pull request trigger to CI workflow#528
dpascualhe merged 1 commit intoJdeRobot:masterfrom
Armaan-Raisinghani:fix/ci

Conversation

@Armaan-Raisinghani
Copy link
Copy Markdown
Contributor

Closes #527
Changes:

  • Added pull_request trigger targeting master branch
  • Removed duplicate Poetry installation step (pip install poetry was redundant since pipx install poetry already runs)

Testing:
The CI YAML is syntactically valid. Effect can be verified by opening a PR after merge.

@dpascualhe
Copy link
Copy Markdown
Collaborator

Thanks to your changes we can see that tests are failing for this PR, so I guess that mission has been accomplished 😄 We will merge and fix tests in another PR. Thanks for your contribution!

@dpascualhe dpascualhe merged commit d7d4393 into JdeRobot:master Apr 17, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Run tests on pull requests, not just master push

2 participants