Skip to content

Add gomodUpdateImportPaths to renovate config#135

Merged
BakerNet merged 1 commit into
mainfrom
dev/renovate-misc
May 20, 2026
Merged

Add gomodUpdateImportPaths to renovate config#135
BakerNet merged 1 commit into
mainfrom
dev/renovate-misc

Conversation

@BakerNet
Copy link
Copy Markdown
Collaborator

Related PR(s)

Related Issue(s)

Summary / Background

@BakerNet BakerNet merged commit 0ab4682 into main May 20, 2026
5 of 6 checks passed
@BakerNet BakerNet deleted the dev/renovate-misc branch May 20, 2026 23:33
@github-actions
Copy link
Copy Markdown

Codeowners approval required for this PR:

@github-actions github-actions Bot requested a review from zbedforrest May 20, 2026 23:33
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the renovate.json configuration to include gomodUpdateImportPaths in the postUpdateOptions. A review comment identifies a potential syntax error in the JSON structure, noting that a closing bracket for a preceding array (likely packageRules) appears to be missing, which would prevent the configuration from being parsed correctly.

Comment thread renovate.json
"prCreation": "immediate"
},
"postUpdateOptions": ["gomodTidy"]
"postUpdateOptions": [
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

critical

The JSON configuration appears to be syntactically invalid. Line 10 ends with a closing brace }, which suggests it is closing an object, but if the preceding section was packageRules (which is an array), it should be closed with a closing bracket ],. This syntax error will prevent Renovate from parsing the configuration file correctly.

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.

1 participant