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

Update TAG Security Documents: README and Charter #1316

Merged
merged 15 commits into from
Jul 31, 2024

Conversation

riaankleinhans
Copy link
Contributor

Fixes: #1312

It PR will work with TOC Repo PR cncf/toc#1376
This PR move the TAG leadership to the TOC repo
It move the TAG Charter to the TOC meeting
Create a file for TAG Chair Emeriti

Copy link

netlify bot commented Jul 10, 2024

Deploy Preview for tag-security ready!

Name Link
🔨 Latest commit 57d67d9
🔍 Latest deploy log https://app.netlify.com/sites/tag-security/deploys/66aa9f6bed3edd0008f9eb0c
😎 Deploy Preview https://deploy-preview-1316--tag-security.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 site configuration.

@riaankleinhans riaankleinhans changed the title [WIP] Update TAG Security Documents: README and Charter Update TAG Security Documents: README and Charter Jul 10, 2024
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@anvega anvega left a comment

Choose a reason for hiding this comment

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

Please address the following changes:

  • Revise the sentimental statement in the README to be more concise and consistent with the overall tone.
  • Move the new file to the community directory and include the gratitude statement there.
  • Update the line about TAG Chairs, Tech Leads, and TOC Liaisons to the improved version provided.

Thank you!

tag-emeritus-leaders.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
riaankleinhans and others added 12 commits July 12, 2024 09:38
Signed-off-by: riaan kleinhans <[email protected]>

md lint

Signed-off-by: riaan kleinhans <[email protected]>

rename file

Signed-off-by: riaan kleinhans <[email protected]>

lint fix

Signed-off-by: riaan kleinhans <[email protected]>

spell file

Signed-off-by: riaan kleinhans <[email protected]>

spell

Signed-off-by: riaan kleinhans <[email protected]>

spell

Signed-off-by: riaan kleinhans <[email protected]>

spell

Signed-off-by: riaan kleinhans <[email protected]>
Signed-off-by: Eddie Knight <[email protected]>
Signed-off-by: Eddie Knight <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
Co-authored-by: Andrés Vega <[email protected]>
Signed-off-by: Riaan Kleinhans <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
Signed-off-by: riaan kleinhans <[email protected]>
@riaankleinhans
Copy link
Contributor Author

@eddie-knight @PushkarJ it there anything blocking this PR from merging?
All I can see it the "link checker" fail because the linked file is created by this PR and does not already exist.

@riaankleinhans
Copy link
Contributor Author

@anvega which PR is blocking this PR? Can I help move that forward? Thanks!

@eddie-knight eddie-knight merged commit 8db0a2d into cncf:main Jul 31, 2024
9 of 10 checks passed
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.

TAG Documentation Update: TAG Security
5 participants