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

build: introduce regal_standalone build flag, use for lint's "fix" hint #1070

Merged

Conversation

srenatus
Copy link
Member

@srenatus srenatus commented Sep 6, 2024

Projects using Regal via its Go module would no longer output the regal fix --help hint.

@@ -68,7 +72,7 @@ checksum:
name_template: "checksums.txt"

snapshot:
name_template: "{{ incpatch .Version }}-next"
version_template: "{{ incpatch .Version }}-next"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

side-track, this was deprecated in some Gorelease version. I don't think we're using snapshot releases, but this will squelch the warning.

build/do.rq Outdated
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My editor did this 🙈 I've taught it to treat .rq files as rego. Yay or nay? I can back out of this change.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change makes the rules look less "task like"... but that's just a matter of time anyway :) So it's good.

Signed-off-by: Stephan Renatus <stephan@styra.com>
Signed-off-by: Stephan Renatus <stephan@styra.com>
@srenatus srenatus force-pushed the sr/fix-command-hint-for-regal-only branch from bf496f5 to 183a9d6 Compare September 6, 2024 08:50
@anderseknert anderseknert merged commit d579c9c into StyraInc:main Sep 6, 2024
4 checks passed
@srenatus srenatus deleted the sr/fix-command-hint-for-regal-only branch September 6, 2024 08:56
srenatus added a commit to srenatus/regal that referenced this pull request Oct 1, 2024
…nt (StyraInc#1070)

Signed-off-by: Stephan Renatus <stephan@styra.com>
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