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

RFC: Add matchFields and matchLabels to top level of blueprints #602

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

jwntrs
Copy link
Contributor

@jwntrs jwntrs commented Feb 10, 2022

Add matchFields and matchLabels to top level of blueprints
Readable

@jwntrs jwntrs added the rfc Requests For Comment label Feb 10, 2022
@netlify
Copy link

netlify bot commented Feb 10, 2022

✔️ Deploy Preview for elated-stonebraker-105904 canceled.

🔨 Explore the source changes: 32895a1

🔍 Inspect the deploy log: https://app.netlify.com/sites/elated-stonebraker-105904/deploys/6220f52f1233b20008c9b14b

@github-actions
Copy link

Hello @jwntrs!, thanks for opening your first Pull Request. Someone will review it soon. Thank you for committing to make Cartographer better

@squeedee
Copy link
Member

One of the asks from the OSS slack channel was for clarity in naming. I think the name or at least title for this PR and for the RFC might be better as
Add MatchFields to top level of blueprints?

@jwntrs jwntrs changed the title Create rfc-0000-blueprint-match-fields.md RFC: Add MatchFields to top level of blueprints Feb 10, 2022
@jwntrs jwntrs changed the title RFC: Add MatchFields to top level of blueprints RFC: Add matchFields and matchLabels to top level of blueprints Feb 16, 2022
@jwntrs jwntrs requested a review from emmjohnson February 25, 2022 16:12
Copy link
Contributor

@cirocosta cirocosta left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Contributor

@sclevine sclevine left a comment

Choose a reason for hiding this comment

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

Needs v1alpha1 schema

Co-authored-by: Ciro S. Costa <[email protected]>
@jwntrs jwntrs removed the breaking label Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rfc Requests For Comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants