-
Notifications
You must be signed in to change notification settings - Fork 522
Update pytest requirement from ~=8.4 to ~=9.0 #1154
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
base: master
Are you sure you want to change the base?
Conversation
Updates the requirements on [pytest](https://github.com/pytest-dev/pytest) to permit the latest version. - [Release notes](https://github.com/pytest-dev/pytest/releases) - [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst) - [Commits](pytest-dev/pytest@8.4.0...9.0.0) --- updated-dependencies: - dependency-name: pytest dependency-version: 9.0.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
|
I am not sure why this fails. The tests work locally and external lichess moves work when lichess-bot is playing locally on my computer. It might be a problem that it is coming from a github actions IP. |
|
The test that fails is contacting chessdb.cn. Since this depends on access to Chinese servers, maybe we should stop this test on GitHub since Chinese access to GitHub can be spotty. Is there a way to do this while still allowing testing on local repositories (like checking |
|
I might be wrong about chessdb.cn. The GitHub Actions logs are ambiguous. We should try making a mock pull request and selectively enabling external move lookups to find the problem. |
|
Further up in the logs, it shows that this is an error with the lichess cloud api. |
|
Also downloading stockfish failed because |
Right. Looks like a 429 error. Should we skip the test if it returns a 429? |
|
Yes, that's a good idea. We could perhaps even remove them as external move apis could change (i.e. add more moves) and I am not sure how good this test actually is. |
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Updates the requirements on pytest to permit the latest version.
Release notes
Sourced from pytest's releases.
... (truncated)
Commits
f4b0fd2Prepare release version 9.0.052d8e68Merge pull request #13889 from bluetech/regendoc-restored6d3e4adoc: fixes for regendoc7cb3974doc: restore missing "# content of pytest.toml" regendoc commands5ae9e47build(deps): Bump django in /testing/plugins_integration (#13881)adb3658Merge pull request #13864 from bluetech/config-cleanups-2a28c08eMerge pull request #13875 from bluetech/ci-tweaksa250954ci: split publish-to-pypi and push-tag jobsebc152fci: update setup python's from 3.11 or 3.* to 3.13dfd796fci: move running update-plugin-list script to toxDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)