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(git): fix exclude files handling in subtree Git repo scan mode #5504

Merged
merged 15 commits into from
Dec 6, 2023

Conversation

vvagaytsev
Copy link
Collaborator

@vvagaytsev vvagaytsev commented Dec 4, 2023

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #5317

Special notes for your reviewer:
Based on test fixes made in #5472.
See individual commits for details.

@vvagaytsev vvagaytsev force-pushed the fix/5317-fix-excluded-files-changes-trigger-rebuild branch from f153957 to feff09f Compare December 4, 2023 14:04
@vvagaytsev vvagaytsev requested review from stefreak and a team December 4, 2023 14:44
@vvagaytsev vvagaytsev marked this pull request as ready for review December 4, 2023 14:44
@vvagaytsev vvagaytsev force-pushed the fix/5317-fix-excluded-files-changes-trigger-rebuild branch from 27d521b to 3783731 Compare December 4, 2023 14:44
@vvagaytsev vvagaytsev marked this pull request as draft December 4, 2023 15:47
@vvagaytsev vvagaytsev changed the title fix(git): don't rebuild container module on excluded files modification fix(git): respect exclude files in Git repo scanning Dec 4, 2023
@vvagaytsev vvagaytsev changed the title fix(git): respect exclude files in Git repo scanning fix(git): respect exclude files config in Git repo scanning Dec 4, 2023
@vvagaytsev vvagaytsev changed the title fix(git): respect exclude files config in Git repo scanning fix(git): fix exclude files handling in subtree Git repo scan mode Dec 5, 2023
@vvagaytsev vvagaytsev force-pushed the fix/5317-fix-excluded-files-changes-trigger-rebuild branch from 497b9b9 to 8eca06b Compare December 5, 2023 10:20
@vvagaytsev vvagaytsev force-pushed the fix/5317-fix-excluded-files-changes-trigger-rebuild branch from 8eca06b to 358850f Compare December 5, 2023 10:45
@vvagaytsev vvagaytsev marked this pull request as ready for review December 5, 2023 10:48
core/src/vcs/git.ts Outdated Show resolved Hide resolved
First, process inclusions, and then exclusions.
For more clarity and consistency in the comments.
@vvagaytsev vvagaytsev added this pull request to the merge queue Dec 6, 2023
Merged via the queue into main with commit 358aeab Dec 6, 2023
46 checks passed
@vvagaytsev vvagaytsev deleted the fix/5317-fix-excluded-files-changes-trigger-rebuild branch December 6, 2023 14:16
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.

Unnecessary config version change can happen when excluded files' contents are modified
2 participants