Closed davidkassa closed 2 years ago
Can you please revisit this. It would be super helpful.
After years ... still no NPM ? :(
Any luck with getting it onto NPM? How can I use this in my project? (Sort of nood with js here, so literally have no clue what to do when dependency is not listed on NPM)
@benmccann would you consider looking into publishing this as a beta (and next
label)?
I'd like to stop using submodules for https://github.com/valor-software/ng2-charts/pull/1276
Opened https://github.com/chartjs/chartjs-chart-financial/pull/95 with the changes required.
@benmccann it's me bothering you again! 😄 What's missing in order to release this?
@santam85 my main hesitation would be putting it up on npm when I'm not really around to support it. Would you have any interest in becoming a maintainer for this?
Yeah sure, happy to help
How do we go about making it happen?
I just invited you to the repo. You'd probably need to coordinate with @etimberg and perhaps @kurkle on the logistics of getting a package published on npm
. I'm not quite sure how that's managed and don't have access to everything required. Hopefully we can set it up on the CI
@etimberg any way of communicating with you/the team other than github PR comments? Any example I should follow in setting up npm release pipelines for this lib?
Sorry about that @santam85, I was away on vacation at the end of august and missed this. The easiest way could be to join the chart.js slack channel and we can communicate there. You could copy the actions from https://github.com/chartjs/Chart.js/ which will automatically use some org specified secrets for deploying to npm.
Package is there: https://www.npmjs.com/package/chartjs-chart-financial
I know that this is not in the npm registry yet. Would you consider adding it as a -beta.x or with a @next tag? It would get more adoption while setting the expectation that it's not ready for prime time. This is a pattern that other libraries, such as vuefire with firestore support, has used.