Skip to content

ci: enable "Poorly Chosen Identifer" lint#4718

Open
johnsaigle wants to merge 2 commits intowormhole-foundation:mainfrom
johnsaigle:ci-static-check-ccq
Open

ci: enable "Poorly Chosen Identifer" lint#4718
johnsaigle wants to merge 2 commits intowormhole-foundation:mainfrom
johnsaigle:ci-static-check-ccq

Conversation

@johnsaigle
Copy link
Copy Markdown
Contributor

No description provided.

@johnsaigle johnsaigle added quality doesn't fix a known bug, doesn't change behavior, but increase code quality ci labels Mar 13, 2026
@johnsaigle johnsaigle changed the title cleanup: enable "Poorly Chosen Identifer" lint ci: enable "Poorly Chosen Identifer" lint Mar 13, 2026
@johnsaigle johnsaigle force-pushed the ci-static-check-ccq branch from 9bb5af2 to d090bfa Compare March 17, 2026 12:33
Copy link
Copy Markdown
Collaborator

@djb15 djb15 left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

networkID string,
bootstrapPeers string,
ethRpcUrl string,
ethRPCURL string,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Imo this is harder to read now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fair, I'll look into renaming this or adding an exception.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

renamed to rpcURL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci quality doesn't fix a known bug, doesn't change behavior, but increase code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants