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

Silence the Activated message #23

Closed
petobens opened this issue May 11, 2023 · 5 comments
Closed

Silence the Activated message #23

petobens opened this issue May 11, 2023 · 5 comments

Comments

@petobens
Copy link
Contributor

I use the statusline to indicate whether I have an active enviroment or not.
I was wondering whether you could make optional the Activation message in

utils.notify("Activated '" .. venv_python)

@linux-cultist
Copy link
Owner

Yes absolutely. I can add it this weekend, but you can also make a PR yourself if you have time before that. Some option that is true by default would probably be easiest to understand for users, and you can set it to false to not get notified.

If you don't have time I will add it on Saturday. :)

@petobens
Copy link
Contributor Author

Hi @linux-cultist ; I'd probably wait for you to add it since I understood per #21 (comment) that you are also going to do some refactoring..

Thanks :)

@linux-cultist
Copy link
Owner

Yeah I want to, but have been very busy now with a new job and lots of stuff to learn there. But I hope I can do some refactoring and cleanup next weekend. But adding that flag is quick and i will do that today. :)

@linux-cultist
Copy link
Owner

linux-cultist commented May 13, 2023

Ok, I added the option notify_user_on_activate that you can set to false, and the plugin will not say anything on venv activation. :) Closing the issue but you can reopen if you have more feedback on this!

@petobens
Copy link
Contributor Author

Great thanks!

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

No branches or pull requests

2 participants