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 #19 #20
Fix #19 #20
Conversation
@@ -57,8 +57,9 @@ sanitize-vars() { | |||
-e "s/export \([A-Za-z_][0-9A-Za-z_]*\)?=/[ -n \"\$\\1\" ] || export \\1=/g" | |||
} | |||
|
|||
while read file; do | |||
for file in $(find-rbenv-vars-files) | |||
do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should follow the current code style and put the do
on the previous line
Hey, @mislav How is my last commit looking? :-) Any comments? |
Looking good, that's exactly how we do it in rbenv. You didn't necessarily have to restore old IFS there since the script is over there and doesn't eval any plugins itself, like rbenv does, so a changed IFS value isn't gonna affect anything. But, if anyone adds more code after that point it's going to save them, so no harm there. |
Kewl, thanks! 😸 |
@mislav, would you be able to merge this in? Or is there a different person to bug? 😈 |
I don't have push access to this repo, but @sstephenson does. |
Fixes #19 Co-authored-by: Mislav Marohnić <git@mislav.net>
Thank you! |
Fixes #19