Skip to content

Conversation

@cubicool
Copy link
Contributor

Pull Request Template

Description

I think this is just another minor misspelling propagated by autocomplete. I'm assuming validateInstanceLayerNames was the intended camel-case to be consistent with everything else?

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

@robertosfield
Copy link
Collaborator

Is the fix simply the capitalization of Layer?

Yes this is correct thing to do so I'll merge, but I can't spot the mis-spelling. I'm mildly dyslexic so there might be a mis-spelling in there which I can't spot...

@cubicool
Copy link
Contributor Author

Yeah, it was just validateInstancelayerNames instead of validateInstanceLayerNames (camel-case per word). If you'd prefer (and if I find any more), I can batch them together into one PR instead of smaller ones?

@robertosfield robertosfield merged commit 1b7d833 into vsg-dev:master Dec 18, 2025
8 checks passed
@robertosfield
Copy link
Collaborator

Separate PRs are fine, the only thing I'd tweak would be the naming of the PR/commit so that if it's spelling typo then label it as such, but if it's just a clean up capitalization then make this clear so I know what to expect. Thanks.

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