Skip to content
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

Fix CI #4387

Closed
wants to merge 1 commit into from
Closed

Fix CI #4387

wants to merge 1 commit into from

Conversation

DimitriPapadopoulos
Copy link
Contributor

@DimitriPapadopoulos DimitriPapadopoulos commented May 23, 2024

Summary of changes

The problem might be that pytest-ruff enforces ruff 0.3.2 and I cannot find a way to force it to use a decently recent version. Why use pytest-ruff in the first place instead of using ruff directly?

platform linux -- Python 3.10.12, pytest-8.2.1, pluggy-1.5.0
cachedir: .tox/py/.pytest_cache
rootdir: /volatile/src/pypa/setuptools
configfile: pytest.ini
plugins: perf-0.15.0, subprocess-1.5.0, enabler-3.1.1, timeout-2.3.1, typeguard-4.2.1, xdist-3.6.1, ruff-0.3.2, cov-5.0.0, mypy-0.10.3, checkdocs-2.13.0, home-0.5.1

Pull Request Checklist

@DimitriPapadopoulos
Copy link
Contributor Author

I remain unable to reproduce the CI failures in other pull requests. Strange.

FURB110 Replace ternary `if` expression with `or` operator
@DimitriPapadopoulos DimitriPapadopoulos force-pushed the fix_CI branch 3 times, most recently from 2d4c974 to 0dca0fa Compare May 23, 2024 17:30
@Avasam
Copy link
Contributor

Avasam commented Jun 17, 2024

Looks like the only change left in this PR is superseded by #4386

@DimitriPapadopoulos
Copy link
Contributor Author

Yes, I have been unable to identify the reason for the CI fails so far. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants