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

Basic docs on extension sets. #58

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from
Open

Conversation

singerb
Copy link
Contributor

@singerb singerb commented Feb 11, 2025

Fixes https://github.com/Automattic/woocommerce.com/issues/22811

This adds some basic documentation on extension sets and their usage. I've put the main page about them under the managed tests, since that's what they apply to at the moment. I've also added a note to the activation test page, since that's the page that people clicking the docs link on an Activation (canonical compatibility result) will arrive at. Finally, I added them to the table of CLI args, with a link to their page, plus a section like the other args with an example command and a bit more info.

ChatGPT turned the list of slugs by marketplace from the code into the markdown list of extension titles and links, which was cool. I clicked through them and it messed up a few of the links, so I've fixed them up manually. Worth double checking them all to make sure the title and link URL are correct.

These are fairly bare bones, so any suggestions on anything that needs expanding would be fine!

@singerb singerb requested a review from a team February 11, 2025 14:34
Copy link
Contributor

@zhongruige zhongruige left a comment

Choose a reason for hiding this comment

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

Links tested great for me and only have a few minor suggestions on the content, thanks for adding in the docs here @singerb!

docs/managed-tests/extension-sets.md Outdated Show resolved Hide resolved
docs/managed-tests/extension-sets.md Outdated Show resolved Hide resolved
docs/managed-tests/extension-sets.md Outdated Show resolved Hide resolved
@singerb
Copy link
Contributor Author

singerb commented Feb 11, 2025

All good catches.

@zhongruige
Copy link
Contributor

Thanks for addressing those!

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.

2 participants