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 this library on npmjs #37

Closed
Nek- opened this issue Aug 5, 2018 · 9 comments
Closed

add this library on npmjs #37

Nek- opened this issue Aug 5, 2018 · 9 comments

Comments

@Nek-
Copy link
Contributor

Nek- commented Aug 5, 2018

Also having a version tag would be nice

@chapterjason
Copy link

chapterjason commented Aug 7, 2018

You can install it without having it on npmjs.

package.json:

// [...]
"dependencies": {
        "@flyover/box2d": "git://github.com/flyover/box2d.ts.git"
}
// [...]

typescript:

import * as box2d from "@flyover/box2d";

@Nek-
Copy link
Contributor Author

Nek- commented Aug 7, 2018

Of course. And by the way, I recommend the following configuration "@flyover/box2d": "github:flyover/box2d.ts#master" which is also compatible with yarn.

But it would be better to have it on npm.

@WowMuchName
Copy link

While you can import the library the ways described above it is hardly a good idea.
If you really want to do it that way, depend on a commit - not master. That way you at least depend on a stable version that wont change in ways that might break your code.

Stumbling over this project I am really impressed with the work that was put into it. However, having it not be properly versioned means I am not going to use it for any serious project.

A 3 year old version of this project is already on npm as 0.0.1. There is no work to be done apart from calling npm publish when the project is updated. This seems like a reasonable time investment, in particular since the entry on npm suggests that the project is dead.

@Nek-
Copy link
Contributor Author

Nek- commented Sep 17, 2018

Indeed, my config suggestion was an example. Anybody should prefer a commit version.

AFAIK there's nothing on npm named "@flyover/box2d". Maybe you speak about node-box2d ( https://www.npmjs.com/package/@flyover/node-box2d ). Notice that:

  • it is not the same project
  • here it is typescript (frontend compliant)

@WowMuchName
Copy link

Yes you are right about it not being the same project. It's just similar named.

@Fxlr8
Copy link

Fxlr8 commented Oct 5, 2018

any updates on this one, @flyover?

@Nek-
Copy link
Contributor Author

Nek- commented Nov 18, 2018

@flyover would you authorize somebody to release on npmjs/yarnpkg ?

@flyover
Copy link
Owner

flyover commented Dec 22, 2018

https://www.npmjs.com/package/@flyover/box2d

@Nek- Nek- closed this as completed Dec 23, 2018
@Nek-
Copy link
Contributor Author

Nek- commented Dec 23, 2018

Thanks @flyover

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

5 participants