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.

Sign up for GitHub

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

Jump to bottom

Stop publishing next #12699

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Stop publishing next #12699

merged 2 commits into from
Jul 12, 2023

Conversation

paul-marechal
Copy link
Member

Related #12519

@paul-marechal paul-marechal added the ci issues related to CI / tests label Jul 11, 2023
@paul-marechal
Copy link
Member Author

We can revert this in the future if it ends up causing trouble. But first we need to try to stop publishing on every commit.

vince-fugnitto
vince-fugnitto approved these changes Jul 12, 2023
View reviewed changes
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

LGTM 👍
The changes make sense and it should help both npm (not to publish so many unused versions), and our CI (not to publish yarn docs for unreleased versions).

@paul-marechal paul-marechal merged commit 39e778a into master Jul 12, 2023
@paul-marechal paul-marechal deleted the mp/stop-publishing-next branch July 12, 2023 16:49
@github-actions github-actions bot added this to the 1.40.0 milestone Jul 12, 2023
@vince-fugnitto vince-fugnitto mentioned this pull request Aug 21, 2023
Open
@jfaltermeier jfaltermeier mentioned this pull request Sep 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vince-fugnitto vince-fugnitto vince-fugnitto approved these changes

@marcdumais-work marcdumais-work Awaiting requested review from marcdumais-work

Assignees
No one assigned
Labels
ci issues related to CI / tests
Projects
PR Backlog
Archived in project
Milestone
1.40.0
Development

Successfully merging this pull request may close these issues.
2 participants
@paul-marechal @vince-fugnitto