Skip to content

Conversation

@iamNoah1
Copy link
Collaborator

update before you start section to remove obsolete information and include info about signing commits.

Describe your changes

Related issue number or link (ex: resolves #issue-number)

see #2017

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

…clude info about signing commits.

Signed-off-by: Noah Ispas (iamNoah1) <[email protected]>
@netlify
Copy link

netlify bot commented May 23, 2025

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 315651a
🔍 Latest deploy log https://app.netlify.com/projects/cncfglossary/deploys/6835e62d02b40f0008371555
😎 Deploy Preview https://deploy-preview-3517--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the lang/en for English label May 23, 2025
@iamNoah1 iamNoah1 added the maintainers Use this label if PR requires maintainers to take action label May 23, 2025
@github-actions
Copy link

To see where this is in the review pipeline and follow the progress, please look at the definition review board.

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

Thanks, @iamNoah1; I have a couple minor suggestions; otherwise, LGTM.

iamNoah1 and others added 2 commits May 23, 2025 17:28
Co-authored-by: Nate W <[email protected]>
Signed-off-by: Noah Ispas <[email protected]>
Co-authored-by: Nate W <[email protected]>
Signed-off-by: Noah Ispas <[email protected]>
Copy link
Collaborator

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@Okabe-Junya
Copy link
Collaborator

Please take a look #3517 (comment) and feel free to merge this PR, thanks again!

Co-authored-by: Nate W <[email protected]>
Signed-off-by: Noah Ispas <[email protected]>
@iamNoah1 iamNoah1 merged commit 821376e into cncf:main May 27, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in CNCF glossary definition review board May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/en for English maintainers Use this label if PR requires maintainers to take action

Projects

Development

Successfully merging this pull request may close these issues.

3 participants