Skip Tests for Packages in src/external_dependencies if Folder Exists#14
Skip Tests for Packages in src/external_dependencies if Folder Exists#14JWhitleyWork merged 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR modifies the GitHub Actions workflow to skip testing packages located in src/external_dependencies to prevent CI failures from external dependencies. The implementation adds conditional logic to check for the existence of this folder and excludes those packages from testing.
Changes:
- Add bash shell configuration to the "Run colcon test" step
- Implement conditional logic to skip packages in
src/external_dependencieswhen running tests
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JWhitleyWork
force-pushed
the
dont-test-external-dependencies
branch
from
e63441d to
93a47d0
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JWhitleyWork
force-pushed
the
dont-test-external-dependencies
branch
from
93a47d0 to
9d3d8e5
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 participants
We shouldn't fail CI jobs because external dependencies fail CI. This checks to see if the workspace contains a
src/external_dependenciesfolder and, if so, skips tests for packages contained in that folder.