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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

publish to GPR as well as NPM! 馃挄馃檶 #450

Closed
wants to merge 22 commits into from
Closed

Conversation

emplums
Copy link

@emplums emplums commented May 16, 2019

Updates our main.workflow to publish to GPR as well as npm :)

This is currently working with the stock npm Action, I want to make it work with primer/publish before merging so we can use canary publishes, etc :)

@shawnbot
Copy link
Contributor

Nice work! Is the plan to have two action blocks (e.g. publish to gpm and publish to npm), or is actions/npm going to support both out of the box?

@emplums
Copy link
Author

emplums commented May 17, 2019

@shawnbot yep that's the idea!

@erictuvesson
Copy link

Hey, any updates on this? 馃槂

@emplums
Copy link
Author

emplums commented Oct 3, 2019

Closing for now as this isn't a super high priority and this branch is super stale at this point!

@emplums emplums closed this Oct 3, 2019
@pqt
Copy link

pqt commented Oct 3, 2019

@erictuvesson I was subscribed to this PR just to see how it went, if you actually just run NPM publish to GPR after primer publish has happened, everything seems to work(ish) as intended with the right versions.

I did it here and it's kind of limping along so I'm happy enough with it for the time being 馃槀 https://github.com/paquette/react-components/blob/master/.github/workflows/main.yml

@emplums emplums deleted the gpr-it-all branch October 22, 2019 19:22
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.

None yet

4 participants