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.

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

Readme example uses v1 as a version reference, but there is no such ref #16

Open
t4lz opened this issue Jul 6, 2023 · 3 comments · May be fixed by #18
Open

Readme example uses v1 as a version reference, but there is no such ref #16

t4lz opened this issue Jul 6, 2023 · 3 comments · May be fixed by #18

Comments

@t4lz
Copy link

t4lz commented Jul 6, 2023

README.md says to use sdkman/sdkman-action@v1, but there is no such tag.

When trying to use the action as specified in README.md, the job fails.

Please consider creating a new release of this action and giving it the v1 tag, or changing the README.md file to use a valid reference.

As a workaround for the meantime, users of this action can specify a branch name like main as a reference, or a full commit hash, like sdkman/sdkman-action@b1f9b696c79148b66d3d3a06f7ea801820318d0f.
@angelcervera
Copy link

Happening the same to me.

@joergi
Copy link

joergi commented Mar 15, 2024

still happening. thanks for the workaround: @t4lz

joergi added a commit to joergi/sdkman-action that referenced this issue Mar 15, 2024
@joergi
Update README.md
0a2486f 
Fixes sdkman#16
@joergi joergi linked a pull request Mar 15, 2024 that will close this issue
Open
@joergi
Copy link

joergi commented Mar 15, 2024

I made a PR for this issue, to fix at least the readme until the tagging is working

#18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update README.md joergi/sdkman-action
3 participants
@angelcervera @joergi @t4lz