Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

remove msgpack pin#818

Merged

Conversation

@prometheanfire
Copy link
Contributor

The hard lock prevents this from being co-installed with many other
packages. For instance, it's preventing it from being included in
openstack (which is on 0.6.2 and working on 1.0.0 now).

Signed-off-by: Matthew Thodemthode@mthode.org


Contributor checklist
  • Builds are passing
  • [×] New tests have been added (for feature additions)

lethee and henningway reacted with thumbs up emoji
The hard lock prevents this from being co-installed with many otherpackages.  For instance, it's preventing it from being included inopenstack (which is on 0.6.2 and working on 1.0.0 now).Signed-off-by: Matthew Thode <mthode@mthode.org>
Copy link
Contributor

@sebito91sebito91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM, thanks for adding!

@sebito91sebito91 merged commitcc41e29 intoinfluxdata:masterApr 11, 2020
ocworld pushed a commit to AhnLab-OSS/influxdb-python that referenced this pull requestApr 13, 2020
The hard lock prevents this from being co-installed with many otherpackages.  For instance, it's preventing it from being included inopenstack (which is on 0.6.2 and working on 1.0.0 now).Signed-off-by: Matthew Thode <mthode@mthode.org>
@cdessez
Copy link

Thanks for merging this fix!

Could we get a new5.3.1 release for this? Currently the latest release ofinfluxdb does not contain the fix and I have to pin it to get my CI pipeline to pass.

Cheers

lethee, tiltec, krtvand, ntkabalan, and henningway reacted with thumbs up emoji

@tiltectiltec mentioned this pull requestJul 12, 2020
derekpierre added a commit to derekpierre/nucypher-monitor that referenced this pull requestSep 10, 2020
There is an issue with influxdb 5.3.0 and msgpack that we are avoiding for now (influxdata/influxdb-python#818).When influxdb 5.3.1 becomes available we should switch to it.
@henningway
Copy link

Thanks for the fix! +1 on tagging a version. We get constant pip error messages because of the version conflict.

@nemesifier
Copy link
Contributor

+1 for a new minor release to fix this

malekcellier and zxytim reacted with thumbs up emoji

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@aviauaviauAwaiting requested review from aviauaviau is a code owner

@xginn8xginn8Awaiting requested review from xginn8xginn8 is a code owner

1 more reviewer

@sebito91sebito91sebito91 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@prometheanfire@cdessez@henningway@nemesifier@sebito91

[8]ページ先頭

©2009-2025 Movatter.jp