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

Do a 1.0.0 release and start following semver #31

Closed
domenic opened this issue Apr 23, 2015 · 11 comments
Closed

Do a 1.0.0 release and start following semver #31

domenic opened this issue Apr 23, 2015 · 11 comments

Comments

@domenic
Copy link

domenic commented Apr 23, 2015

It would be great to be able to depend on getting feature updates and bugfixes in jsdom. Since the module is below version 1.0.0 right now we have no idea what will change in each release.

@stash-sfdc
Copy link
Contributor

I think the API is pretty stable and philosophically I agree with you.

Are there any features or API you consider missing? (cc @inikulin)

@stash-sfdc stash-sfdc self-assigned this Apr 23, 2015
@stash-sfdc
Copy link
Contributor

#9 is the only API-breaking change I'd like to make.

@stash-sfdc
Copy link
Contributor

cc @lalitkapoor

@lalitkapoor
Copy link
Contributor

I agree, I think this has been pretty stable for a while and happy to see it follow semver. I'd like to look at 2 bugs, but definitely not preventing a v1.0.0 release.

@inikulin
Copy link
Contributor

@lalitkapoor #19 was fixed with #30.
@stash-sfdc The thing that bothers me most is the strict mode. If enabled it doesn't pass IETF's tests, therefore currently non-strict mode is stricter than "strict" mode. So, I suggest to get rid of "strict" mode. I can issue a PR if you are OK with it.

@stash-sfdc
Copy link
Contributor

@inikulin 👍 if it actually isn't compliant, I'm OK with that. It will reduce a lot of complexity, both from the code and the API. If you don't have time, let me know and I'll take it on. :)

@inikulin
Copy link
Contributor

@stash-sfdc I'll take care of it at Monday.

@stash-sfdc
Copy link
Contributor

@inikulin ok, see #34

@stash-sfdc stash-sfdc added this to the 1.0 API Changes milestone Apr 24, 2015
@stash-sfdc
Copy link
Contributor

OK, just waiting for #16 now - there's a PR in the pipeline.

@stash-sfdc
Copy link
Contributor

Er, wait, that doesn't need to be fixed for 1.0.x. I'll cut the release soon.

@stash-sfdc
Copy link
Contributor

[email protected] published to NPM.

domenic added a commit to jsdom/jsdom that referenced this issue Apr 29, 2015
Now that it's stable (salesforce/tough-cookie#31) this floating version specifier should serve us well for a while.
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

4 participants