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

Use latest golang versions #81

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

larhauga
Copy link
Contributor

No description provided.

@larhauga larhauga force-pushed the bump-golang-version branch 2 times, most recently from 6c95afa to d307531 Compare November 20, 2023 10:10
@anderseknert
Copy link
Member

Can you rebase this and fix the conflict? That'd be a good first test of publishing the edge tag I guess :)

Signed-off-by: Lars Haugan <lars.haugan@sparebank1.no>
@anderseknert
Copy link
Member

anderseknert commented Nov 20, 2023

Screenshot 2023-11-20 at 11 20 15

Did you see that on main? I'm not sure what it's referring to tbh 🤔 Is it from the cleanup phase or just delayed to make it look like so?

@larhauga
Copy link
Contributor Author

Did you see that on main? I'm not sure what it's referring to tbh 🤔 Is it from the cleanup phase or just delayed to make it look like so?

I think I messed up the repo+tag part here. Something got mangeled due to the -v2 part, but the main issue is a missing variable (TAG_NAME i think):

docker push foo:
invalid reference format

Signed-off-by: Lars Haugan <lars.haugan@sparebank1.no>
@larhauga
Copy link
Contributor Author

I think this branch should contain the necessary changes for the workflows to work correctly

@anderseknert anderseknert merged commit 6d0e11a into open-policy-agent:main Nov 20, 2023
4 checks passed
@anderseknert
Copy link
Member

Seems to have worked! It's marked as "private" though. I'll need to look into why, and how to change it.

@larhauga larhauga deleted the bump-golang-version branch November 20, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants