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

Backport of resource: enforce consistent naming of resource types into release/1.16.x #17870

Conversation

hc-github-team-consul-core
Copy link
Collaborator

Backport

This PR is auto-generated from #17611 to be assessed for backporting due to the inclusion of the label backport/1.16.

The below text is copied from the body of the original PR.


Description

For consistency, resource type names must follow these rules:

  • Group must be snake case, and in most cases a single word.
  • GroupVersion must be lowercase, start with a "v" and end with a number.
  • Kind must be pascal case.

These were chosen because they map to our protobuf type naming conventions.


Overview of commits

@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/boxofrad/normalize-resource-type/urgently-accurate-drake branch 2 times, most recently from e73916e to a436e41 Compare June 26, 2023 12:25
@hc-github-team-consul-core hc-github-team-consul-core enabled auto-merge (squash) June 26, 2023 12:25
Copy link
Collaborator

Choose a reason for hiding this comment

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

Auto approved Consul Bot automated PR

@vercel vercel bot temporarily deployed to Preview – consul June 26, 2023 12:31 Inactive
@hc-github-team-consul-core hc-github-team-consul-core merged commit 5c71d7e into release/1.16.x Jun 26, 2023
@hc-github-team-consul-core hc-github-team-consul-core deleted the backport/boxofrad/normalize-resource-type/urgently-accurate-drake branch June 26, 2023 12:40
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.

3 participants