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

Add NPM distribution #563

Merged
merged 7 commits into from
Oct 27, 2023
Merged

Conversation

TheSpyder
Copy link
Contributor

Closes #561

Check the following

  • Keep 100% code coverage
  • Be properly formatted
  • Documentation changes are included
  • Include a change file if expected

@miniscruff
Copy link
Owner

miniscruff commented Oct 27, 2023

https://www.npmjs.com/package/changie ran the commands locally and published the package, the npx and install commands seem to be working for me on linux, not sure if you want to give this a try before I merge it.

Edit: I have also added the NPM token and with some changes ready to go we should be able to auto release v1.15 soon with NPM support.

@TheSpyder
Copy link
Contributor Author

yes, that works - sorry, it took a while to install from Australia as it hadn't reached the CDN end nodes yet 😅

@miniscruff
Copy link
Owner

yes, that works - sorry, it took a while to install from Australia as it hadn't reached the CDN end nodes yet 😅

haha, that explains the timing of all your comments 😏

Alright, lets see how this goes. 😆

@miniscruff miniscruff merged commit 0b09647 into miniscruff:main Oct 27, 2023
@TheSpyder
Copy link
Contributor Author

hehe yeah I'm sure some of my activity is at weird hours for you 😆

I don't think it's a huge problem, but you seem to have used the main branch to build and release that rather than the 1.14 tag. This build includes the swap from changie merge --project to auto-merging all projects.

@miniscruff
Copy link
Owner

miniscruff commented Oct 27, 2023

hehe yeah I'm sure some of my activity is at weird hours for you 😆

I don't think it's a huge problem, but you seem to have used the main branch to build and release that rather than the 1.14 tag. This build includes the swap from changie merge --project to auto-merging all projects.

oh yea... let me delete v1.14 from NPM as its not accurate then.
Edit: Not sure its possible to delete a release...

@miniscruff
Copy link
Owner

@TheSpyder ok it seems like v1.15.0 is up on NPM properly with your auto deployment. Had some issues with winget but all good for now, will have to come back to that one later.

@TheSpyder
Copy link
Contributor Author

Ah yeah NPM doesn't really support deleting versions, the closest they have is deprecate

But nobody is using it yet so I'm sure they won't notice 🙂

@TheSpyder TheSpyder deleted the 561-npm-publishing branch December 8, 2023 06:59
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.

NPM distribution
2 participants