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 table of contents in Readme file #107

Merged
merged 2 commits into from
Oct 11, 2022
Merged

Update table of contents in Readme file #107

merged 2 commits into from
Oct 11, 2022

Conversation

gallacher
Copy link
Contributor

Description

Updated the table of contents in the README.md file to remove the following links:

  • Branching
  • Maintainers

The contents of the branching document contents have been moved inside the contributor guide. The maintainers document is no longer needed. Instead, code owners are being tracked inside the code owners file: https://github.com/dell/csm/blob/main/.github/CODEOWNERS.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
N/A

Checklist:

  • Have you verified your document update?

How Has This Been Tested?

I have verified the document update and ensured there are no broken links

@gallacher gallacher merged commit e6b9c6b into main Oct 11, 2022
@gallacher gallacher deleted the update-readme branch October 11, 2022 16:35
jooseppi-luna pushed a commit that referenced this pull request Oct 13, 2022
jooseppi-luna added a commit that referenced this pull request Oct 14, 2022
* e2e tests working and improved

* Update README.md

* add modules field in test spec

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* fixed gofmt added cleanup

* Update README.md

* Update allowedlist.yaml

* Update allowedlist.yaml

* Update values.yaml

* e2e tests working and improved

* add modules field in test spec

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* fixed gofmt added cleanup

* Update README.md

* Update allowedlist.yaml

* Update allowedlist.yaml

* Bump up version to 1.0.0 (#108)

* Removed links from TOC (#107)

* Updated the go version to 1.19 from 1.18 (#109)

* updated from v1alpha1 to v1 (#110)

* Update values.yaml

* observability

Co-authored-by: rensyct <80810999+rensyct@users.noreply.github.com>
Co-authored-by: gallacher <35462391+gallacher@users.noreply.github.com>
Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com>
ChristianAtDell added a commit that referenced this pull request Oct 15, 2024
ChristianAtDell added a commit that referenced this pull request Oct 15, 2024
* e2e tests working and improved

* Update README.md

* add modules field in test spec

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* fixed gofmt added cleanup

* Update README.md

* Update allowedlist.yaml

* Update allowedlist.yaml

* Update values.yaml

* e2e tests working and improved

* add modules field in test spec

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* fixed gofmt added cleanup

* Update README.md

* Update allowedlist.yaml

* Update allowedlist.yaml

* Bump up version to 1.0.0 (#108)

* Removed links from TOC (#107)

* Updated the go version to 1.19 from 1.18 (#109)

* updated from v1alpha1 to v1 (#110)

* Update values.yaml

* observability

Co-authored-by: rensyct <80810999+rensyct@users.noreply.github.com>
Co-authored-by: gallacher <35462391+gallacher@users.noreply.github.com>
Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com>
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