Skip to content

fix path to script/try-publish #634

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
Merged
merged 2 commits into from
Jan 8, 2019
Merged

fix path to script/try-publish #634

merged 2 commits into from
Jan 8, 2019

Conversation

@shawnbot
Copy link
Contributor

@shawnbot shawnbot commented Jan 7, 2019

The #625 build on master failed. I didn't change the script/try-publish path, but this should fix it not being found.

@shawnbot
Copy link
Contributor Author

shawnbot commented Jan 8, 2019

Welp, after the build failed I tried to publish the changes in #625 with script/release on this branch, but I can't publish primer-progress because I don't have permission. @broccolini, do you have the keys for the primer-css npm user, and if so could you add primer-progress to the developers team packages? 🙏

@shawnbot shawnbot requested a review from jonrohan January 8, 2019 00:06
@shawnbot shawnbot requested a review from emplums January 8, 2019 00:08
@shawnbot shawnbot merged commit 7eb7374 into master Jan 8, 2019
@shawnbot shawnbot deleted the release-hotfix branch January 8, 2019 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonrohan jonrohan Awaiting requested review from jonrohan

1 more reviewer

@emplums emplums emplums approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@broccolini broccolini

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shawnbot @emplums @broccolini