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

Cutting releases v1.9 and v2 #98

Closed
edorivai opened this issue Sep 4, 2018 · 10 comments
Closed

Cutting releases v1.9 and v2 #98

edorivai opened this issue Sep 4, 2018 · 10 comments

Comments

@edorivai
Copy link
Collaborator

edorivai commented Sep 4, 2018

Hey @mjackson 👋

tl;dr

  • I think master is ready to be published as 1.9.
  • Could you please review the changes in next?
  • If you've given your 👍 for next, I'll go and polish the docs

A bit of an update on the current state of the repo. I've merged most of the outstanding PRs into either master or next. I think master should be publishable as a minor version bump, as we've only added Typescript definitions (#76) and the onChange callback (#95). Docs look good, and I've prepared the CHANGES.md, you just have to add the release date.

I don't think next is ready for deployment yet, as I'd like to do a cleanup of the docs. We've accumulated some new features over time, and while they've been added to the docs, I feel the docs now lack cohesion. Moreover, there's an outstanding PR which should improve the state of SSR (mainly preventing nasty edge-case bugs).

After we complete those things on next, we could release a v2-alpha. What do you think?

@flq
Copy link

flq commented Oct 1, 2018

Hi there,
would have loved the TS signatures, but it looks like a lot of good things are outstanding (last release is 8 months ago) - have you create some alternative package where I can pick up the latest additions?

@edorivai
Copy link
Collaborator Author

edorivai commented Oct 3, 2018

@flq I have not created a fork - if that is what you mean. As it stands releases are done by @mjackson, who's a busy man 😅. I guess you could fork this repo for yourself if you really have to. Otherwise, if the TS typings are all you're after, you can just copy/paste them into your own repo, or you could even create an entry in DefinitelyTyped if you feel like it.

@flq
Copy link

flq commented Oct 3, 2018

I’m sure he’s busy. At the end I settled for react-responsive, which pretty much seems to do the same - I mean this can also always be a viable possibility, to stop supporting one lib in favor of another that does the same such that efforts can be focused.

@tstirrat15
Copy link
Contributor

I'd love to see v2 released as well.

@gersomvg
Copy link

A minor version bump shouldn't be too much work right? Would be nice to have the Safari event listener bug solved, which is merged a month ago, @mjackson .

@mjackson
Copy link
Member

Hey @edorivai, I think I forgot to mention that I fully automated releases on this repo :) I will take a look at the changes and cut some new releases today, but let's sync up on how this is to be done going forward. I don't want to be the bottleneck here :)

@edorivai
Copy link
Collaborator Author

@mjackson Sounds good, I'll shoot you an email so we can take the conversation there.

@edorivai
Copy link
Collaborator Author

@grsmvg, @flq, 1.9 has been published to npm, it includes TS signatures and a fix for our safari bug. Sorry for the delay 😅.

@tstirrat15 progress on v2 will be tracked in #110

@gersomvg
Copy link

Great, thanks a bunch! :)

@gersomvg
Copy link

gersomvg commented Dec 12, 2018

I get a bunch of runtime errors with version 1.9.0

Error: Cannot find module "@babel/runtime/helpers/esm/inheritsLoose"

Module not found: Error: Can't resolve '@babel/runtime/helpers/esm/assertThisInitialized' in ...

Module not found: Error: Can't resolve '@babel/runtime/helpers/esm/defineProperty' in ...

Module not found: Error: Can't resolve '@babel/runtime/helpers/esm/inheritsLoose' in ...

I'll create an issue for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

5 participants