-
Notifications
You must be signed in to change notification settings - Fork 6
feat: include uninstallation logic #70
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
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
12eb748
refactor: added ScriptConfigurationEntry
alestiago 000aa81
refactor: remove empty space
alestiago ac0a310
refactor: use completion installation to verify installation
alestiago 4421964
refactor: removing unused code
alestiago 8e6939c
feat: reformatted start and end comments
alestiago 3397c16
refactor: used barrel file
alestiago 6d37050
test: refactor expectation
alestiago c9cca0a
test: remove white space
alestiago ba6eaaf
Merge branch 'main' into refactor/script-configuration-entry
alestiago ac810c3
feat: fixed merge issue
alestiago 98f4cf0
test: removed extra space
alestiago ccf8de3
feat: uninstall logic
alestiago 111892c
Merge branch 'main' into refactor/uninstallation-logic
alestiago fd16cfa
test: removeFrom
alestiago 2b19a14
fix: added break condition
alestiago 817ae1b
refactor: used while instead of do while
alestiago c87eb66
feat: refined removedFrom
alestiago b571c14
feat: refined uninstall logic
alestiago e8335c9
feat: tested and configured uninstalling behaviour
alestiago c327c2c
test: included missing executable script deletion
alestiago bfccc39
feat: refined uninstallation logic
alestiago 11b990e
feat: improved uninstall behaviour for multiple shells
alestiago b2db32a
test: added CompletionUnistallationException
alestiago ac3a066
docs: improved uninstall documentation
alestiago 55ee7ec
test: improved names
alestiago e249a97
refactor: renamed executablName to rootCommand
alestiago 96b2796
refactor: made shouldDelete false by default
alestiago db0b752
docs: fixed wrong documentation
alestiago cba7174
test: renamed file
alestiago 0736ca5
test: include _test in test name
alestiago bc847d4
Update lib/src/installer/completion_installation.dart
alestiago da11c75
Merge branch 'main' into refactor/uninstallation-logic
renancaraujo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.