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

docs: Organize and expand contributing documentation #253

Merged
merged 15 commits into from
Jun 17, 2024

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Jun 14, 2024

Relates to p1718394299821639/1718026506.388569-slack-C06DRMD6VPZ.

Proposed Changes

Organize and expand documentation to capture helpful guidance and information
for contributors.

Testing Instructions

Navigate various links found within the README file and other documentation
files.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@dcalhoun dcalhoun added the [Type] Documentation Improvements or additions to documentation label Jun 14, 2024
@dcalhoun dcalhoun self-assigned this Jun 14, 2024
@dcalhoun dcalhoun marked this pull request as ready for review June 14, 2024 19:44
@dcalhoun dcalhoun requested review from a team June 14, 2024 19:44
Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

Choose a reason for hiding this comment

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

The changes look good. The instructions are clear and well presented and the document is well structured.

Copy link
Contributor

@fluiddot fluiddot left a comment

Choose a reason for hiding this comment

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

LGTM 🎊 ! Thanks @dcalhoun for improving and expanding the documentation 🙇 !

docs/code-contributions.md Outdated Show resolved Hide resolved
dcalhoun and others added 3 commits June 17, 2024 08:27
Contributors may not be familiar with the Calypso project or its purpose.

Co-authored-by: Carlos Garcia <[email protected]>
Calypso is not associated with the open-source WordPress project.
Avoid potential confusion from missing or outdated dependencies by
recommending that contributors always install the latest dependencies
before building the source code.
@dcalhoun dcalhoun merged commit f431f55 into trunk Jun 17, 2024
10 checks passed
@dcalhoun dcalhoun deleted the docs/organize-and-expand-contributing-documentation branch June 17, 2024 12:51
@dcalhoun
Copy link
Member Author

Merged these changes with the feedback and approval provided thus far. 🚀

For context and posterity, I completed this reorganization as I felt the README file reached a "tipping point" where it was quite verbose and unapproachable. The merged changes were inspired by WordPress/gutenberg's documentation.

That said, anyone should feel welcome to further critque and/or reorganize the documentation. Documentation organization and tone is always quite subjective. I welcome others to continue proposing changes, as any documentation needs to evolve alongside its project as the scope and complexity increases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants