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

Where is the tag for 1.2.8? #74

Closed
willdurand opened this issue Nov 27, 2018 · 11 comments
Closed

Where is the tag for 1.2.8? #74

willdurand opened this issue Nov 27, 2018 · 11 comments

Comments

@willdurand
Copy link

Hi, I see that a version 1.2.8 is listed on npm but I cannot find the tag here. Could you push it by chance? Thanks.

@FND
Copy link
Collaborator

FND commented Nov 28, 2018

That would have to be 4fc9daf, as that's the only commit between v1.2.7 and when I started contributing towards v2.0.0 - but I'll leave it to @andrew to confirm that.

@andrew
Copy link
Member

andrew commented Nov 28, 2018

I have no idea 😂

@andrew
Copy link
Member

andrew commented Nov 28, 2018

I’d recommend using 2.X

@willdurand
Copy link
Author

I have no idea 😂

oh, heh.

I’d recommend using 2.X

Yep, but our test suite fails when upgrading from 1.2.8 to 2.0.0 (using the legacy API). It seem an invalid data to decode does not throw any error anymore and we get NaN instead. I am not sure though.

@andrew
Copy link
Member

andrew commented Nov 28, 2018

Maybe it's worth changing the title of the issue to report that bug instead?

@willdurand
Copy link
Author

I'll report the bug once I'll have a test case, in another issue. This issue is valid though, it'd be nice to have tags for each npm release.

@FND
Copy link
Collaborator

FND commented Nov 28, 2018

Thank you, @willdurand. Perhaps this is also related to #57 (though the behavior shouldn't have changed in that regard).

@andrew
Copy link
Member

andrew commented Nov 28, 2018

@willdurand as I mentioned before, I don't know where to put that tag as it was a long time ago.

@andrew andrew closed this as completed Nov 28, 2018
@andrew andrew added the wontfix label Nov 28, 2018
@FND
Copy link
Collaborator

FND commented Nov 28, 2018

@andrew: I've just checked against the contents of the npm package and I'm fairly certain that the commit I'd identified above was in fact v1.2.8 - do you mind if I create that tag retroactively?

@andrew
Copy link
Member

andrew commented Nov 28, 2018

@FND sure

@FND
Copy link
Collaborator

FND commented Nov 28, 2018

Done: https://github.com/andrew/base62.js/commits/v1.2.8

Hope that helps, @willdurand.

@FND FND removed the wontfix label Nov 28, 2018
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

3 participants