FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-16379: Fix SQLite version checks in test_module_constants() by erlend-aasland · Pull Request #28809 · python/cpython · GitHub

/ cpython Public

bpo-16379: Fix SQLite version checks in test_module_constants() - #28809

Merged
miss-islington merged 1 commit into
python:mainfrom
erlend-aasland:sqlite-fix-version-checks
Oct 7, 2021
Merged

bpo-16379: Fix SQLite version checks in test_module_constants()#28809
miss-islington merged 1 commit into
python:mainfrom
erlend-aasland:sqlite-fix-version-checks

Conversation

erlend-aasland commented Oct 7, 2021
edited by miss-islington
Loading

Copy link
Copy Markdown
Contributor

https://bugs.python.org/issue16379

Automerge-Triggered-By: GH:pablogsal

Copy link
Copy Markdown
Contributor Author

@pablogsal Can I reuse issue 16379 for this, or is the skip issue label ok here?

Copy link
Copy Markdown
Contributor Author

GH-27786 (which introduced these tests) was run multiple times with the buildbots. There is apparently too little variation of SQLite versions in the buildbots.

Copy link
Copy Markdown
Member

@pablogsal Can I reuse issue 16379 for this, or is the skip issue label ok here?

You can reuse the issue if you wish :)

pablogsal changed the title Fix SQLite version checks in test_module_constants() bpo-16379: Fix SQLite version checks in test_module_constants() Oct 7, 2021

Copy link
Copy Markdown
Contributor Author

Thanks, Pablo :)

Copy link
Copy Markdown
Contributor

@erlend-aasland: Status check is done, and it's a success ✅ .

miss-islington merged commit 8deb7af into python:main Oct 7, 2021
erlend-aasland deleted the sqlite-fix-version-checks branch October 7, 2021 21:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL