search

aslafy-z / helm-git

Helm Plugin - Install Helm Charts strait from Git repositories
Apache License 2.0
287 stars 46 forks source link

fix: _uri_scheme: is read only #248

Closed jkroepke closed 1 year ago

jkroepke commented 1 year ago

In helm-secrets, I'm running the CI with posh shell.

On that shell, I'm getting error like this while running helm-git:

_uri_scheme: is read only

This PR should fixes this.

aslafy-z commented 1 year ago

Mmh, weird I didn't catched it. Thank you!

jkroepke commented 1 year ago

The posh is not a shell for operator or end-users. The feature set of the posh shell is really low, however it tries to be 100% POSIX compliant. As I know, the Debian maintainers are using the POSH shell to validate the shell script for POSIX correctness. Thats, why I'm using that shell on my CI.

aslafy-z commented 1 year ago

I added some linking with the BusyBox shell inside the CI for this reason, however it looks like it may be not working as expected. I will open an issue to revise that when I get some time. I did not know POSH, will have a look. Thank you for the quick fix!

aslafy-z commented 1 year ago

:tada: This PR is included in version 0.15.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: