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

Fix minor typos and file name in contributing docs #1651

Merged
merged 4 commits into from
Sep 20, 2023

Conversation

seadanda
Copy link
Contributor

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context,
including:

  • What does this PR do?

This PR fixes two minor typos in the contributing top level docs, and fixes a broken link to another file.

  • Why are these changes needed?

While reading the guidelines I noticed two small typos and a broken link to the documentation guidelines in the same directory.

  • How were these changes implemented and what do they affect?

Both typos were fixed. The file has been renamed from DOCUMENTATION_GUIDELINE.md to DOCUMENTATION_GUIDELINES.md to match the link, as the link seems correct vs the filename. There are no other references to this file within this repo.

There are no open issues related to this

DOCUMENTATION_GUIDELINES.md is referred to in CONTRIBUTING.md, but the
file is wrongly named. No other refs to the old name exist in the repo.
@seadanda seadanda added A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). T11-documentation This PR/Issue is related to documentation. labels Sep 20, 2023
@ggwpez ggwpez enabled auto-merge (squash) September 20, 2023 18:10
@ggwpez ggwpez merged commit 41e38b2 into master Sep 20, 2023
9 checks passed
@ggwpez ggwpez deleted the donal-fix-contributing-docs branch September 20, 2023 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). T11-documentation This PR/Issue is related to documentation.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants