search

cyanogilvie / Tcl.js

Javascript implementation of the Tcl language
Other
157 stars 6 forks source link

NPM package? #4

Open drom opened 4 years ago

drom commented 4 years ago

Is it possible to install this code via NPM?

cyanogilvie commented 4 years ago

Not at this stage. I've never packaged a module for node so I don't know what would be required.

On Fri, Jan 3, 2020, 22:19 Aliaksei Chapyzhenka notifications@github.com wrote:

Is it possible to install this code via NPM?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cyanogilvie/Tcl.js/issues/4?email_source=notifications&email_token=AAZHPTABRJ2K5N7CYIWS4HTQ36MVJA5CNFSM4KCRQBOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ID6I7RQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZHPTGPGLND3SNIE5YWSQ3Q36MVJANCNFSM4KCRQBOA .

drom commented 4 years ago

It will require conversion of you amd style files into require style. Something like this: https://github.com/drom/Tcl.js/commit/1882dab692219752113909e9a616b10c870ef037 I can help if you wish.

cyanogilvie commented 4 years ago

I forgot that it was actually possible to use it in node, as the tools in static_tools/ do (by using a node version of require.js). I've ported everything to ES6 style modules now though, which is supported in both modern browsers and node >= 12. See the static_tools/*.mjs scripts for examples using it from node.

If you want to write the glue to make it available through npm, I'd be interested in merging it, as long as it doesn't interfere with browser use.

drom commented 4 years ago

Good move on porting everything to ES6 modules. It is the future. I will try to package it for NPM. I also want to make it work for https://observablehq.com

rbucker commented 4 years ago

OMG do not do it. if you have to find a way to package the code... maybe you can sub the project and add some scripts. node and npm are poisoned