Skip to content
This repository was archived by the owner on Jul 16, 2023. It is now read-only.

Support latest analyzer 5.10.0 #1216

Closed
asaarnak opened this issue Apr 3, 2023 · 7 comments
Closed

Support latest analyzer 5.10.0 #1216

asaarnak opened this issue Apr 3, 2023 · 7 comments

Comments

@asaarnak
Copy link

asaarnak commented Apr 3, 2023

image

@asaarnak
Copy link
Author

asaarnak commented Apr 3, 2023

Our analyzer update is blocked:
image

@asaarnak
Copy link
Author

asaarnak commented Apr 3, 2023

Seems like there is a pull request already: #1214

@asaarnak
Copy link
Author

asaarnak commented Apr 3, 2023

Maybe we should use ^5.1.0 instead of ">=5.1.0 <5.9.0" in this dependency as it shouldn't be breaking change?

@incendial
Copy link
Member

Maybe we should use ^5.1.0 instead of ">=5.1.0 <5.9.0" in this dependency as it shouldn't be breaking change?

It's not possible due to the analyzer not following the semver. Sometimes breaking changes can appear in minor releases.

@incendial
Copy link
Member

Available in 5.7.1

@asaarnak
Copy link
Author

Maybe we should use ^5.1.0 instead of ">=5.1.0 <5.9.0" in this dependency as it shouldn't be breaking change?

It's not possible due to the analyzer not following the semver. Sometimes breaking changes can appear in minor releases.

Ok, thanks! How often this happens?

@incendial
Copy link
Member

Have no idea, but what matters is that it happens

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

No branches or pull requests

2 participants