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

rebar3 dependency "git_subdir" tag not supported for tag (rule: protocol_for_deps) #370

Closed
kivra-pauoli opened this issue Nov 5, 2024 · 4 comments · Fixed by #373
Closed
Labels

Comments

@kivra-pauoli
Copy link
Contributor

Bug Description

#343 is not complete, in that it doesn't support tag where it supports branch.

To Reproduce

Take the test file on that pull request and replace branch with tag, here.

Expected Behavior

No function clause.

rebar3 Logs

Not applicable.

Additional Context

  • OS: all
  • Erlang version: all
  • rebar3 version: all
@paulo-ferraz-oliveira
Copy link
Collaborator

I looked at rebar3's implementation, and also the documentation. Most likely I'll also pull request there to update https://github.com/erlang/rebar3/blob/main/rebar.config.sample.

@paulo-ferraz-oliveira paulo-ferraz-oliveira changed the title rebar3 dependency "git_subdir" tag not support for tag rebar3 dependency "git_subdir" tag not supported for tag Nov 10, 2024
@paulo-ferraz-oliveira
Copy link
Collaborator

rebar3 and rebar3.org have been pull requested for update. Going to import my knowledge here, now...

@paulo-ferraz-oliveira paulo-ferraz-oliveira changed the title rebar3 dependency "git_subdir" tag not supported for tag rebar3 dependency "git_subdir" tag not supported for tag (rule: protocol_for_deps) Nov 10, 2024
@paulo-ferraz-oliveira
Copy link
Collaborator

@elbrujohalcon, I wonder if the rule should be more permissive and allow for git:// alongside https://.

@elbrujohalcon
Copy link
Member

@elbrujohalcon, I wonder if the rule should be more permissive and allow for git:// alongside https://.

I think that's a good idea.

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

Successfully merging a pull request may close this issue.

3 participants