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

chore(tests): tie usage output to snapshots #3246

Closed
wants to merge 0 commits into from

Conversation

wraithgar
Copy link
Member

We have another PR in flight that will start auto-generating some
content in docs/content/commands, and will run postsnap, which means
that if we tie the usage output to a snapshot, the docs will also be
updated when the snapshots change.

References

Related to #3243

@wraithgar wraithgar requested a review from a team as a code owner May 14, 2021 14:06
@wraithgar wraithgar force-pushed the gar/test-usage-snapshot branch 2 times, most recently from 5c20371 to 15721c9 Compare May 14, 2021 15:37
@darcyclarke darcyclarke added Release 7.x work is associated with a specific npm 7 release semver:patch semver patch level for changes release: next These items should be addressed in the next release labels May 17, 2021
@darcyclarke darcyclarke requested a review from isaacs May 20, 2021 16:49
isaacs pushed a commit that referenced this pull request May 20, 2021
We have another PR in flight that will start auto-generating some
content in `docs/content/commands`, and will run `postsnap`, which means
that if we tie the usage output to a snapshot, the docs will also be
updated when the snapshots change.

PR-URL: #3246
Credit: @wraithgar
Close: #3246
Reviewed-by: @isaacs
@isaacs isaacs closed this May 20, 2021
@wraithgar wraithgar deleted the gar/test-usage-snapshot branch November 2, 2021 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: next These items should be addressed in the next release Release 7.x work is associated with a specific npm 7 release semver:patch semver patch level for changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants