Skip to content

Update couchbase API to 3.0.0#6321

Merged
auvipy merged 5 commits into
masterfrom
couchbase_3
Aug 31, 2020
Merged

Update couchbase API to 3.0.0#6321
auvipy merged 5 commits into
masterfrom
couchbase_3

Conversation

@thedrow

@thedrow thedrow commented Aug 30, 2020

Copy link
Copy Markdown
Contributor

Description

Supersedes #6094.

@thedrow thedrow changed the title *Note*: Before submitting this pull request, please review our [contributing guidelines](https://docs.celeryproject.org/en/master/contributing.html). Update couchbase API to 3.0.0 Aug 30, 2020
@lgtm-com

lgtm-com Bot commented Aug 30, 2020

Copy link
Copy Markdown

This pull request introduces 1 alert and fixes 2 when merging 92abb65 into 5ab26db - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 2 for Non-exception in 'except' clause

@thedrow thedrow mentioned this pull request Aug 30, 2020
23 tasks
@auvipy
auvipy self-requested a review August 31, 2020 08:13
@thedrow

thedrow commented Aug 31, 2020

Copy link
Copy Markdown
Contributor Author

Something is very wrong with the memcached container. I'm investigating.

EDIT:
It seems that the newest version of env has gained a new option -S which is present in focal but not in bionic.

We need to test in bionic anyway since PyPy is not yet available on focal and focal is not currently a stable environment in Travis.

@lgtm-com

lgtm-com Bot commented Aug 31, 2020

Copy link
Copy Markdown

This pull request fixes 2 alerts when merging f2d0716 into 5fb1393 - view on LGTM.com

fixed alerts:

  • 2 for Non-exception in 'except' clause

@auvipy
auvipy merged commit 5806976 into master Aug 31, 2020
@thedrow

thedrow commented Aug 31, 2020

Copy link
Copy Markdown
Contributor Author

@auvipy Are we OK with the Windows build failing due to PYCBC-1018?

jeyrce pushed a commit to jeyrce/celery that referenced this pull request Aug 25, 2021
* Update couchbase API to 3.0.0

* Code cleanup.

* Revert back to bionic.

* Actually bump the minimum required version to 3.0.0.

* Make the is-memcached-running script more portable between Ubuntu versions.

Co-authored-by: Mathieu Chataigner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants