feat(sync): add .gitattributes export-ignore management to dev-tools:sync (#13)#27
Open
feat(sync): add .gitattributes export-ignore management to dev-tools:sync (#13)#27
Conversation
- Extract GitAttributesCommand from SyncCommand for standalone execution - SyncCommand now calls gitattributes via runCommand like gitignore - Register GitAttributesCommand in DevToolsCommandProvider - Update tests to include new command and dependencies
coisa
force-pushed
the
feature/13-gitattributes-sync
branch
from
82ea6cd to
500f85b
Compare
… rules Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>
…ter implementations - Added Reader and ReaderInterface for reading .gitattributes files. - Implemented Merger and MergerInterface to handle merging export-ignore entries. - Created Writer and WriterInterface for writing normalized .gitattributes content. - Added ExportIgnoreFilter to manage paths to be ignored during export. - Developed tests for Reader, Merger, Writer, and ExportIgnoreFilter functionalities. - Updated GitAttributesCommand to utilize new Reader, Merger, and Writer classes. - Enhanced existing tests to cover new functionality and ensure proper behavior. Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>
- Add full list of folders/files from issue #13 to CandidateProvider - Add gitattributes command documentation to specialized-commands.rst - Update README.md with gitattributes command and table entry - Update SyncCommand docs to mention gitattributes call
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.gitattributesexport-ignore management as part ofdev-tools:syncGitAttributesnamespace with separate concerns for maintainabilityNew Classes
CandidateProviderInterface/CandidateProviderExistenceCheckerInterface/ExistenceCheckerMergerInterface/MergerTesting
Behavior
The sync command now:
Closes #13