-
Notifications
You must be signed in to change notification settings - Fork 30
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
Improve CLI output name handling by adding file extensions #2914
Merged
MW-Friedrich
merged 41 commits into
main
from
fix/2800/improving-cli-output-name-by-adding-file-extensions
Aug 17, 2022
Merged
Changes from 13 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
9641a68
FileExtensionHandler added + Tests #2800
charlotteKehm f1ae74a
solution for path problem #2800
charlotteKehm 6ccc508
case: path without filename considered #2800
charlotteKehm 047b54d
CodeMaatImporter + CSVImporter fixed #2800
charlotteKehm 53c0a66
RawTextParser + TokeiImporter fixed #2800
charlotteKehm 8693581
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich 269c8ec
minor refactoring to prevent files from being created while testing #…
charlotteKehm 8f99e18
pipeline fehler #2800
charlotteKehm dff5cab
write jsons in terminal for golden_test.sh #2800
charlotteKehm d2dbc3c
updated changes with review remarks #2800
charlotteKehm 8bf7570
fix test #2800
charlotteKehm be64b7a
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
charlotteKehm 0e4dc9a
remove test boolean #2800
charlotteKehm 7dd2bee
remove comment, sysout variable
Chrisp485 f9e46f7
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich e25446e
refacotor OutputFileHandler.kt, adjust parsers
Chrisp485 0be9773
Merge branch 'fix/2800/improving-cli-output-name-by-adding-file-exten…
Chrisp485 a97416d
refactor RawText Parser and adjust tests
Chrisp485 a1169a3
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich 1baf638
remove sysout from golden_test.sh
Chrisp485 76cfe80
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich 5e52669
Add tests to GitLogParser and SVNLogParser
MW-Friedrich c9aa2aa
Refactor SVNLogParser.kt for linter correctness
MW-Friedrich 5656ec1
Fix file ending for CrococosmoImporter
MW-Friedrich d03e872
Fix path-separator not correctly added in OutputFileHandler
MW-Friedrich c9d88a2
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich 845ea45
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich 2cdaec7
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich af8c3b9
Update CHANGELOG.md
MW-Friedrich da8671a
Merge remote-tracking branch 'origin/fix/2800/improving-cli-output-na…
MW-Friedrich 5085376
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich e420dca
Fix EdgeFilter and MergeFilter not using the new OutputFileHandler
MW-Friedrich bd180ab
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich dce89fc
Merge remote-tracking branch 'origin/fix/2800/improving-cli-output-na…
MW-Friedrich e6d79cc
Refactor a test for linter happiness
MW-Friedrich 864bcd0
Add a test for the EdgeFilter
MW-Friedrich afefb61
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich c1ee009
Merge remote-tracking branch 'origin/fix/2800/improving-cli-output-na…
MW-Friedrich b7b28ff
Refactor test for MergeFilter to JUnit
MW-Friedrich 88ad7dd
Fix StructureModifier not using new OutputFileHandler
MW-Friedrich 817b464
Merge branch 'main' into fix/2800/improving-cli-output-name-by-adding…
MW-Friedrich File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
21 changes: 14 additions & 7 deletions
21
...someImporter/src/main/kotlin/de/maibornwolff/codecharta/importer/jasome/JasomeImporter.kt
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this.