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(installer/pwsh): fixes some details on installer and uninstaller #2404

Merged
merged 1 commit into from Apr 2, 2022

Conversation

guerra08
Copy link
Contributor

Description

This PR fixes and issue that prevents a new line to be correctly added on the Powershell $Profile file to set the lvim alias.

This also fixes and issue that prevents the execution of the uninstall.ps1 script. It was lacking the call to the main function. It also fixes some issues when trying to parse the cli args, and also when trying to test some of the old folders to be removed.

Test process

This was tested on a local install of LunarVim, in a Windows 11 machine with the latest version of Powershell installed.

@guerra08
Copy link
Contributor Author

Updated commit message

@guerra08
Copy link
Contributor Author

guerra08 commented Mar 30, 2022

Updated commit message

@kylo252 Needed to remove upper case on subject, sorry

Copy link
Collaborator

@kylo252 kylo252 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thank you so much!

@kylo252 kylo252 changed the title fix(installer/pwsh): Fixes some details on installer and uninstaller fix(installer/pwsh): fixes some details on installer and uninstaller Apr 2, 2022
@kylo252 kylo252 merged commit 84e499a into LunarVim:rolling Apr 2, 2022
@guerra08
Copy link
Contributor Author

guerra08 commented Apr 3, 2022

LGTM, Thank you so much!

Happy to help!

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.

None yet

2 participants