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

Feat/don't ask unneeded input on (de)activate #57

Merged

Conversation

AntPeixe
Copy link
Contributor

I've also added --follow-symlinks to the sed commands.

A lot of people use dotfile repos and create symlinks. I think it makes more sense to have the original changed than to break the link (the backup is still created).

I was making changes to my .zshrc in my repo and nothing was being picked up as the link broke, it was quite confusing .

If all plugins are active, trying to active shouldn't do anything.
Similarly for deactive and remove.

Fixes zap-zsh#53
A lot of people use dotfile repos and create symlinks, follow them and
change source instead of breaking the link
@ChristianChiarulli
Copy link
Member

looks good, pls fix merge conflicts

@AntPeixe
Copy link
Contributor Author

Done 👍

Really strange conflict as it was complaining on a line I didn't touch. I solve it with merge, but if we prefer rebase let me know and I'll adapt that.

@ChristianChiarulli ChristianChiarulli merged commit ddb7cd3 into zap-zsh:master Nov 13, 2022
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