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

Fix absolute/relative path handling in --affected #92

Merged
merged 1 commit into from
Oct 13, 2024

Conversation

elliotgoodrich
Copy link
Owner

The conditional was incorrectly reversed due to my misreading of std::error_code so we never attempted to look up the absolute or relative path for paths passed in to --affected.

It was a bit tricky to do unit test for absolute paths as they need to be generated beforehand. This commit is less lazy and includes unit tests for absolute and relative path handling by generating files in the CMake generation step.

The conditional was incorrectly reversed due to my misreading of
`std::error_code` so we never attempted to look up the absolute or
relative path for paths passed in to `--affected`.

It was a bit tricky to do unit test for absolute paths as they need to
be generated beforehand.  This commit is less lazy and includes unit
tests for absolute and relative path handling by generating files in the
CMake generation step.
@elliotgoodrich elliotgoodrich merged commit f3a3d3e into main Oct 13, 2024
8 checks passed
@elliotgoodrich elliotgoodrich deleted the fix-abs-rel-condition branch October 13, 2024 14:23
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.

1 participant