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

CLEANUP toYaml #4

Closed
wants to merge 2 commits into from
Closed

Conversation

mattclegg
Copy link

This also seems to work

goreleaser/goreleaser#5068 (comment)

@qrkourier
Copy link
Member

Ah, nice. I'll try this. I didn't see your suggestion before it merged to upstream with the 'if ne then true; else false' way.

@qrkourier
Copy link
Member

Also, the conditionals we added in .goreleaser.yml are redundant since the magefile.go avoids running the docker step for the same condition. Still, doesn't hurt anything to express it both places.

@qrkourier
Copy link
Member

superseded by #6

@qrkourier qrkourier closed this Sep 26, 2024
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.

2 participants