Skip to content

Conversation

@rxmarbles
Copy link
Member

No description provided.

@msluther msluther mentioned this pull request Dec 17, 2020
package.json Outdated
"version": "6.0.0",
"description": "The approach to JavaScript at GoDaddy. We think it's pretty decent.",
"scripts": {
"prepublishOnly": "npm test",

Choose a reason for hiding this comment

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

I'm not sure this will do much since we don't publish at the top level but instead on an individual package basis.

Choose a reason for hiding this comment

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

Good callout. Is it possible for this to be placed in the sub-packages instead?

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we do this in a separate PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fair enough I'll do a separate PR to script it out better for publishing and such

@rxmarbles rxmarbles merged commit 03392c6 into master Dec 18, 2020
@rxmarbles rxmarbles deleted the rm/migrate-to-actions branch December 18, 2020 19:11
wcole1-godaddy pushed a commit that referenced this pull request Oct 13, 2025
* [dist] add workflow and update devDeps

* [dist] clean up lock files of private registry

* [tiny] add prepublishOnly for protection

* [tiny] revert prepublish
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.

7 participants