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 guidance and tooling for documentation #1431

Merged
merged 17 commits into from
Jan 16, 2024

Conversation

ADubhlaoich
Copy link
Contributor

@ADubhlaoich ADubhlaoich commented Dec 29, 2023

Proposed changes

This commit adds a README file to the /site subfolder instructing a reader on how to build and contribute to the documentation. It introduces the tooling, explains the Makefile, then details the conventions around Hugo and Markdown usage.

Addresses #1430 .

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

This commit adds a README file to the /site subfolder instructing a
reader on how to build and contribute to the documentation. It
introduces the tooling, explains the Makefile, then details the
conventions around Hugo and Markdown usage.
@ADubhlaoich ADubhlaoich requested review from a team as code owners December 29, 2023 17:09
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 29, 2023
@ADubhlaoich
Copy link
Contributor Author

Pushed a second commit to address the linting issues. I think attention should be drawn to the subfolder README somewhere in the CONTRIBUTING file in the base of the repo but it's unclear where.

Maybe appending the /site/ folder for docs underneath "Project Structure", and the README under "Development Guide"?

Copy link
Contributor

@kate-osborn kate-osborn left a comment

Choose a reason for hiding this comment

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

@ADubhlaoich do we still need this developer doc for documentation?

site/README.md Outdated Show resolved Hide resolved
site/README.md Outdated Show resolved Hide resolved
site/README.md Outdated Show resolved Hide resolved
@kate-osborn kate-osborn requested a review from a team January 2, 2024 18:48
This commit incorporates some feedback to the README file while also
updating the Makefile to follow suit. There were some options listed
that didn't actually exist and others that weren't listed.
@ADubhlaoich
Copy link
Contributor Author

Incorporated the feedback from the PR thus far, which is also reflected with changes to the Makefile itself.

I have one more pass to make to merge the existing documentation README's contents into this one, at which point it should be reviewed again.

site/README.md Outdated Show resolved Hide resolved
@ADubhlaoich
Copy link
Contributor Author

pre-commit.ci autofix

@ADubhlaoich ADubhlaoich changed the title Add documentation README to /site subfolder Update guidance and tooling for documentation Jan 8, 2024
@ADubhlaoich ADubhlaoich self-assigned this Jan 8, 2024
CONTRIBUTING.md Outdated Show resolved Hide resolved
site/README.md Show resolved Hide resolved
site/README.md Show resolved Hide resolved
site/Makefile Outdated Show resolved Hide resolved
@ADubhlaoich
Copy link
Contributor Author

@kate-osborn Any amendments? There's been other approvals.

@ADubhlaoich ADubhlaoich enabled auto-merge (squash) January 16, 2024 14:33
@ADubhlaoich ADubhlaoich merged commit 3cadadb into nginxinc:main Jan 16, 2024
26 checks passed
@ADubhlaoich ADubhlaoich deleted the docs/add-documentation-readme branch January 16, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants