Redis.jl icon indicating copy to clipboard operation
Redis.jl copied to clipboard

Time for a new release?

Open david-macmahon opened this issue 4 years ago • 9 comments

Since PR #81 has been merged into master, it would be nice to have that fix/functionality available as a tagged release rather than having to use Redis.jl#master to get it. I think it is technically breaking since it changes the value passed to subscription handler functions, so maybe v1.1.0?

david-macmahon avatar Jul 14 '21 01:07 david-macmahon

I just noticed that Project.toml on master already specifies the version as v1.1.0 so I guess it just needs to be tagged as such.

david-macmahon avatar Jul 14 '21 01:07 david-macmahon

Hi David,

I'm happy to tag a new release. I think the process for this changed within the past year or so (maybe earlier) and I have not been actively keeping up with the Julia ecosystem since then. Is this something that can be resolved with a PR or should I be looking into setting up some new CD for the repo?

jkaye2012 avatar Jul 20 '21 14:07 jkaye2012

I think the breaking change means that it should be v2.0.0 - any package with a compat field set to 1 or 1.1 could break if the version only goes up to 1.1.0.

The easiest way to register now is to use @JuliaRegistrator and @JuliaTagBot.

It may be worth thinking if there are any other breaking changes that may want to be done, and lumping them in together?

EDIT: Sorry I made the registrator comment!

mmiller-max avatar Sep 03 '21 13:09 mmiller-max

Error while trying to register: Action not recognized: and

JuliaRegistrator avatar Sep 03 '21 13:09 JuliaRegistrator

Error while trying to register: Action not recognized: and

JuliaRegistrator avatar Sep 03 '21 13:09 JuliaRegistrator

I'm unaware of any pending changes that would necessitate a major version bump. #76 is the only open feature request right now, which would be additive and only a minor bump.

jkaye2012 avatar Sep 13 '21 13:09 jkaye2012

I was thinking that #81 technically changes the public API (subscription callback function now receives different inputs to before), and therefore needs a major version bump as it's a breaking change.

mmiller-max avatar Sep 14 '21 07:09 mmiller-max

Sorry, yes I agree with that. I meant there aren't any others pending.

On Tue, Sep 14, 2021, 1:50 AM Mal Miller @.***> wrote:

I was thinking that #81 https://github.com/JuliaDatabases/Redis.jl/pull/81 technically changes the public API (subscription callback function now receives different inputs to before), and therefore needs a major version bump as it's a breaking change.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/JuliaDatabases/Redis.jl/issues/82#issuecomment-918901882, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7FL7B4T3WZG6WT7H5HKLLUB35GDANCNFSM5AKNTEVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

jkaye2012 avatar Sep 14 '21 11:09 jkaye2012

Ah of course, sorry for misunderstanding

mmiller-max avatar Sep 15 '21 07:09 mmiller-max

Bump on this; is there something we can help with?

Thanks!

pankgeorg avatar Apr 08 '23 17:04 pankgeorg

If you'd like to tag/upload the release, I would be happy to accept a PR for this

jkaye2012 avatar Apr 10 '23 17:04 jkaye2012