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 network documents #1395

Merged
merged 7 commits into from
Aug 27, 2024
Merged

Update tag network documents #1395

merged 7 commits into from
Aug 27, 2024

Conversation

devbird007
Copy link
Contributor

This PR fixes cncf/tag-network#42

Along with work in TAG Network PR.

Hi @riaankleinhans, I'd like to complete the table for TAG Network in the tags/cncf-tags.md file, but I do not know how to get the Organization and Term of the TAG Network Chairs and Tech Lead.

Signed-off-by: Emmanuel Taiwo <[email protected]>
Signed-off-by: Emmanuel Taiwo <[email protected]>
Signed-off-by: Emmanuel Taiwo <[email protected]>
@riaankleinhans
Copy link
Contributor

@devbird007 thank you for the update. Here is some additional updated I would like you to do.

About the tags/cncf-tags.md file

The term & organization is not implemented by all TAGs, so it is OK to leave those fields blank.
Please go ahead and add markdown tables around all the TAGs, same as you did for TAG Network.

About the TAG Network charter file

The change you made to the charter file is correct. Fixing the hyperlink and removing the leadership, meeting and contact information is inline with the issue.

Additionally, please look at this PR in the TAG Network repo
tags/cncf-tags.md
Ignore the changes from line 103 as you already removed that as discussed above.
Bring over the changes in that PR and include it in this copy of the charter that you are working on. The aim is to delete the charting the the TAG Network repo once this copy of the chart is up to date.

When deleting the projects listed, replace the landscape hyperlink with this filtered link:
https://landscape.cncf.io/?group=projects-and-products&view-mode=grid&tag=network

Thank you.

tags/cncf-tags.md Outdated Show resolved Hide resolved
@devbird007
Copy link
Contributor Author

Hi @riaankleinhans, in your comment under About the TAG Network charter file. I notice you made a mistake in the PR you are linking in the TAG Network Repo, because when I click the link, it just loops right back to this PR.

Might this be the PR you actually wanted to reference: here.

If so, I will make the changes accordingly now.

@riaankleinhans
Copy link
Contributor

Hi @riaankleinhans, in your comment under About the TAG Network charter file. I notice you made a mistake in the PR you are linking in the TAG Network Repo, because when I click the link, it just loops right back to this PR.

Might this be the PR you actually wanted to reference: here.

If so, I will make the changes accordingly now.

You are correct. Changes look good.
I will leave it open for approval by the TAG Network leads and liaison.

@riaankleinhans
Copy link
Contributor

/cc @nicholasjackson @ZackButcher @leecalcote

Can you please review and +1. This would also address #34
Next steps we would help update the TAG Network README and remove the duplicate copy of the charter in the TAG repo.

tags/cncf-tags.md Outdated Show resolved Hide resolved
Signed-off-by: Riaan Kleinhans <[email protected]>
@riaankleinhans riaankleinhans merged commit 7e81a9d into cncf:main Aug 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
process-documentation Doc changes for process and procedures tag-network
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TAG Documentation Update: TAG Network
7 participants