You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
Once thing I forgot: Not sure if it makes sense to use a major version > 0 if recast itself is 0.x. Maybe @benjamn wants to finally release a major version too? ;)
The reason will be displayed to describe this comment to others. Learn more.
If you feel strongly about this, sure ;)
4.0 would be a bit silly, but I tend to avoid 1.0 releases if I've had 0.x releases for a long period of time, as it makes people think it's the very first version. I normally skip from 0.x directly to 2.0 if 0.x is relatively stable and there's lots of major changes or refactoring.
d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this should have been a minor bump (0.4) since there's a new public API (the
testUtils
), so it's not purely a bugfix release.d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When < 1.0.0, the patch number is increased for new features and bugfixes. The minor version is only increased for breaking changes.
Of course we can also consider to make jscodeshift 1.0 ;)
d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, that's something I wasn't aware of. Thanks for the info!
We could follow React's lead and make the next version jscodeshift 4 rather than 0.4 😛
d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you feel strongly about this, sure ;)
Actually, it seems I'm not quite correct: http://semver.org/#spec-item-4
But what I mentioned before is the pattern I follow. So yeah, we could also have done
0.4.0
.d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once thing I forgot: Not sure if it makes sense to use a major version > 0 if recast itself is 0.x. Maybe @benjamn wants to finally release a major version too? ;)
d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure! I think this
ast-types
issue is the biggest to-do in the way of arecast
/ast-types
1.0: benjamn/ast-types#57d07ac8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4.0 would be a bit silly, but I tend to avoid 1.0 releases if I've had 0.x releases for a long period of time, as it makes people think it's the very first version. I normally skip from 0.x directly to 2.0 if 0.x is relatively stable and there's lots of major changes or refactoring.