-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release docs? #139
Comments
I didn't write anything up, but the primary changes are:
|
Thank you. FYI moving methods feels like a breaking change and the version
bump should have probably been a major version change. Otherwise as a
consumer of this lib, blindly updating the version would break our code.
On Wed, Dec 21, 2022 at 9:00 AM Silas Sewell ***@***.***> wrote:
I didn't write anything up, but the primary changes are:
- Switched from callbacks to promises (the pre-1.x version supported
both, the new version only supports Promises)
- Removed support for legacy versions of node (basically only the
currently maintained versions are officially supported, although older
versions might still work)
- Enabled http keep-alive by default
- Removed lock (this was always marked as experimental and I wasn't
confident in the implementation)
- Moved various methods from consul.acl to consul.acl.legacy
- Update to papi 1.x
—
Reply to this email directly, view it on GitHub
<#139 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADWTFQJ6SPHGAHAXGNSN4TWOMLRZANCNFSM6AAAAAATDTUZSA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Tom Caflisch
***@***.***
|
Woops, I assumed you meant the the If you mean the |
Got it. Thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed there was a new release but wanted to see what was changed in the current release. Are there any release docs for this lib?
The text was updated successfully, but these errors were encountered: