3.1.0 #611
3.1.0 #611
Conversation
| @@ -1,5 +1,8 @@ | |||
| # Changelog | |||
|
|
|||
| ## 3.1.0 | |||
| * NEW: Added TypeScript declaration file for compatibility with TypeScript projects. See: https://github.com/getsentry/raven-js/pull/610 | |||
There was a problem hiding this comment.
There are obviously more changes here.
My first impression was "why are we doing a big version bump just for adding TS declarations", then realized there are lots of things.
There was a problem hiding this comment.
Isn't the bigger breaking change that we remove the ability to fetch context from the client?
There was a problem hiding this comment.
We already removed that in 3.0.0 – it's in the changelog. We just removed extra extraneous code that we missed.
There was a problem hiding this comment.
My first impression was "why are we doing a big version bump just for adding TS declarations", then realized there are lots of things.
If "big" is "minor version", then yes. It's not a bug fix / patch. It's a forward-compatible API change (if you use TypeScript).
There was a problem hiding this comment.
Yeah, I'm torn. That, to me, falls under something like, us adding a feature into a plugin. Does that mandate a minor version bump for something that affects a small % of users? I dunno.
There was a problem hiding this comment.
That, to me, falls under something like, us adding a feature into a plugin.
That should be a minor bump too, yeah. It's an argument for moving these into their own repo.
|
😐 |
|
😅 |
cc @mattrobenolt