Skip to content

Move more of version check to Rego#1513

Merged
anderseknert merged 1 commit intomainfrom
rego-version-check
Apr 30, 2025
Merged

Move more of version check to Rego#1513
anderseknert merged 1 commit intomainfrom
rego-version-check

Conversation

@anderseknert
Copy link
Copy Markdown
Member

Just something little I wanted to try :)

Just something little I wanted to try :)

Signed-off-by: Anders Eknert <anders@styra.com>
Comment thread pkg/linter/linter.go
// the supplied configuration. This makes use of the linter rule settings
// to produce a single list of the rules that are to be run on this linter
// instance.
func (l Linter) DetermineEnabledRules(ctx context.Context) ([]string, error) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The changes in this file are only due to golangcilint now saying public methods should precede private ones

Copy link
Copy Markdown
Contributor

@charlieegan3 charlieegan3 left a comment

Choose a reason for hiding this comment

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

LGTM!

@anderseknert anderseknert merged commit bebff27 into main Apr 30, 2025
5 checks passed
@anderseknert anderseknert deleted the rego-version-check branch April 30, 2025 10:11
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