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
@michaelbull Because of #110 we are locked to v1.x of this library. Would you consider publishing a new 1.x build which removes the @Deprecated warnings on OK and Err? If we're happy using 1.1.21, which we are, it would be nice not to have to suppress that warning everywhere we use those APIs.
We can't downgrade to 1.1.20, pre-deprecation, because we're using some of the newer features in 1.1.21.
The text was updated successfully, but these errors were encountered:
Hi Graham, sorry to hear that the Kotlin team has still not yet prioritised the issue that's stopping your upgrade.
I am open to making a new, unadvertised, 1.1.22 release that simply undeprecates them, but I won't be pushing people to stay on 1.x as I have no plans to maintain it. I'm also unsure how I'm going to do this cleanly as the CI workflow is set up to publish only on changes to master. Maybe if I branch the 1.1.21 release and change the workflow to include the new branch name it'll work - not sure.
As an aside, what happens if you typealias the deprecated methods? I am wondering if they still inherit their underlying type's deprecated nature or does it remove the deprecated warnings?
@michaelbull Because of #110 we are locked to v1.x of this library. Would you consider publishing a new 1.x build which removes the
@Deprecated
warnings onOK
andErr
? If we're happy using 1.1.21, which we are, it would be nice not to have to suppress that warning everywhere we use those APIs.We can't downgrade to 1.1.20, pre-deprecation, because we're using some of the newer features in 1.1.21.
The text was updated successfully, but these errors were encountered: