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

Update npm package to support Gen 9 #503

Closed
olanmccarthy opened this issue Dec 18, 2022 · 7 comments
Closed

Update npm package to support Gen 9 #503

olanmccarthy opened this issue Dec 18, 2022 · 7 comments

Comments

@olanmccarthy
Copy link

Hey guys, just tried downloading the npm package for @smogon/calc and it currently doesn't support generation 9.
It seems to be working fine in this project so was wondering when you are going to release the updated package?

@BCleveland
Copy link

Same issue here, does anyone know how I could get around this for the time being and have Node use a local version of this repo instead?

@monsanto
Copy link
Member

The reason the NPM package hasn't been updated is because we don't use the NPM package internally for anything, so over time the process to distribute it decays. Whoever was passionate about maintaining it moved on with life, and the only time we notice is when someone makes a GH issue about it.

I'm not opposed to releasing an NPM package of any of the repos in this organization, but it has to be zero overhead for our organization. So that probably means no meaningful semver (0.x?), and an automated build & upload process as part of GH actions. Or it could be a git branch on here that has the results of a build, which would save us the trouble of maintaining a different set of credentials. If someone wants to work on that then you have my blessing and thanks.

@JonatasBSM
Copy link

The reason the NPM package hasn't been updated is because we don't use the NPM package internally for anything, so over time the process to distribute it decays. Whoever was passionate about maintaining it moved on with life, and the only time we notice is when someone makes a GH issue about it.

I'm not opposed to releasing an NPM package of any of the repos in this organization, but it has to be zero overhead for our organization. So that probably means no meaningful semver (0.x?), and an automated build & upload process as part of GH actions. Or it could be a git branch on here that has the results of a build, which would save us the trouble of maintaining a different set of credentials. If someone wants to work on that then you have my blessing and thanks.

Hey, monsanto! I hope you're doing well. Npm package situation still the same?

@monsanto
Copy link
Member

@JonatasBSM yes it is

@JonatasBSM
Copy link

@JonatasBSM yes it is

I see. Thank for the answer!

@ajhyndman
Copy link
Contributor

ajhyndman commented Mar 13, 2023

I did have a look into automating npm releases this weekend. There is some good documentation with examples using GitHub Actions. My takeaway though is that there is still a bare minimum manual intervention required. The version number for each module as published must be reflected in the package.json file before publication.

Therefore, someone will need to at least bump the committed version number in the repository to trigger each release.

With that said, the directions listed in this repository don't involve much more than that manual version bump, so I'm not sure there's much value to add with automation.

Instead, I have tried my hand at publishing a fork here:

https://www.npmjs.com/package/@ajhyndman/smogon-calc

There are no changes in this fork. I just pulled the latest code from @smogon/damage-calc and published it under another name.

If anyone wants to update the official publication, that would still be preferred, of course! The process seems pretty straightforward for someone with access to the right credentials.

@thejetou
Copy link
Collaborator

thejetou commented Oct 6, 2023

v0.8.0 has since been released with support for Generation 9. I'll try my best to keep the package updated.

@thejetou thejetou closed this as completed Oct 6, 2023
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

No branches or pull requests

7 participants
@monsanto @ajhyndman @olanmccarthy @thejetou @BCleveland @JonatasBSM and others