Semver compliance #3843
HaakonSvane
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hey people 👋
First of all, thanks for creating such an awesome package. We have been using this for quite some time in production with our Expo app and it has been awesome.
I am often puzzled by how hard it is to manage version compatability for Expo apps. New versions of Expo SDK versions come with a range of deps to upgrade to specific versions to ensure stability. Part of why this is difficult is that package versions sometimes don't follow semantic versioning.
I bring this up here after noticing a breaking change in the interface of this package after the 2.2.1 release. More specifically, a the
onLayoutprop was removed. I stumbled across this issue when upgrading our app to the latest Expo framework, which crossed the 2.2.1 release. Found out about this after reading an issue registered in the react native vision camera repo. Some users were bothered by this as well.Writing just to hear your thoughts on how breaking changes should / are communicated for this package. Semver compliance requires this type of changes to be communicated with a major upgrade, and I see no mentions of the breaking change in the 2.2.1 release either
Beta Was this translation helpful? Give feedback.
All reactions