You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
pre-commit
[INFO] Installing environment for git://github.com/markdownlint/markdownlint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('/usr/local/bin/bash', '/Users/abuxton/.rbenv/shims/gem', 'build', 'mdl.gemspec')
return code: 1
expected return code: 0
stdout: (none)
stderr:
rbenv: version `default' is not installed (set by RBENV_VERSION environment variable)
i have tried, ensuring local rbenv, I have tried pre-commit clean/uninstall and reinstall to no success, mdl will brew install mdl successfully and works fine.
ah yeah, this has regressed because it's difficult to test! pre-commit ~correctly identified that you're using shims globally so it shouldn't necessarily trust that ruby, but then used it anyway 😆 -- this if here needs to also apply to 'default':
actually hmm this seems really strange -- pre-commit should still be creating an rbenv shim in that case so my suggested patch above isn't quite correct
Now that this has been merged in, when is the next planned package release?
if you look at the release history, nothing is "planned", I maintain this in my free time :) if you or your company depend on this package I'd recommend sponsoring
Activity
[-]Rbenv default ruby issue on hook install[/-][+]Rbenv default ruby issue on install[/+]asottile commentedon Nov 17, 2020
ah yeah, this has regressed because it's difficult to test! pre-commit ~correctly identified that you're using shims globally so it shouldn't necessarily trust that ruby, but then used it anyway 😆 -- this
ifhere needs to also apply to'default':pre-commit/pre_commit/languages/ruby.py
Line 43 in 59acc7e
asottile commentedon Nov 17, 2020
actually hmm this seems really strange -- pre-commit should still be creating an rbenv shim in that case so my suggested patch above isn't quite correct
asottile commentedon Nov 17, 2020
@abuxton can you try out the patch in #1700 -- it fixed the case I tried locally but I'm not 100% it solves your usecase
georgealton commentedon Nov 18, 2020
I had exactly the same issue today in AWS CodeBuild.
Tried the fix #1700 and it solves it for me.
pbr0ck3r commentedon Nov 20, 2020
Now that this has been merged in, when is the next planned package release?
asottile commentedon Nov 20, 2020
if you look at the release history, nothing is "planned", I maintain this in my free time :) if you or your company depend on this package I'd recommend sponsoring