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

Make sourcing .bash_profile and .bashrc optional/disableable #233

Closed
demosdemon opened this issue Mar 12, 2015 · 3 comments
Closed

Make sourcing .bash_profile and .bashrc optional/disableable #233

demosdemon opened this issue Mar 12, 2015 · 3 comments

Comments

@demosdemon
Copy link

While researching the cause of #215, I see that both .bash_profile and .bashrc are loaded. cli-beautify.coffee:43

It's not necessary in my issue with autopep8 since I already provide the path for autopep8.

Another issue from sourcing those two files, I have my .bash_profile set up to do as it pleases and .bashrc set up to source .bash_profile. So, in this case, .bash_profile is being sourced twice unnecessarily causing some significant delay (I put too much in mine).

I would like it if this were at least disableable from the configuration if not fully customizable.

@Glavin001
Copy link
Owner

Absolutely agree. This will be implicitly resolved with #164 , however that may very well be a while off. At least this is a temporary fix.

I am accepting pull requests for this issue.

@Glavin001
Copy link
Owner

A quick fix that would be even better would be having users set their $PATH for Atom Beautify. That was the whole point of that code you referenced.

@Glavin001
Copy link
Owner

Published to v0.25.0

This is a very big release with lots of internal changes and (hopefully) no breaking changes. Please let me know if you are experiencing any unusual behaviour / breaking changes after updating. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants