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.

Sign up for GitHub

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

Jump to bottom

fix: correct handling of repository string #63

Open

andrestylianos wants to merge 1 commit into slipset:master from andrestylianos:fix/repository-string

andrestylianos commented Feb 6, 2024

Hit the following error when trying to use 0.2.2 in CI

nth not supported on this type: Character

I reproduced it locally and the reason is that line 222 uses options instead of the opts that have already been processed.

This means that when :repository is a string such as "repo" we end up with

(->> options
     :repository ;; => "repo"
     (into []) ;; => [\r \e \p \o]
      first) ;; => \r

which it then tries to destructure with [exec-args-repo-id exec-args-repo-settings].


          fix: correct handling of repository string

5975d7a

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

Labels

None yet