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

🐛 Use compatible pytest < 8 when running MyPy against Python 3.7 #937

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

Jamim
Copy link
Contributor

@Jamim Jamim commented Feb 1, 2024

Hello @webknjaz,

What do these changes do?

These changes:

  • fix pytest/mypy compatibility issue for Python 3.7
  • bump pypa/cibuildwheel to 2.16.5

Are there changes in behavior for the user?

Issues on CI will be resolved.

Related issue number

Checklist

  • I think the code is well written

Best regards!

@Jamim
Copy link
Contributor Author

Jamim commented Feb 1, 2024

Do we need a news fragment for these trivial fixes? I don't think so.

@Jamim
Copy link
Contributor Author

Jamim commented Feb 1, 2024

@webknjaz, as you can see, this PR fixes CI failures. The only reason for the ❌ mark is the Timeline protection which, I believe, might be skipped in this case.

@webknjaz
Copy link
Member

webknjaz commented Feb 1, 2024

Please, make atomic PRs. The one with testing fixes would need a contrib change note. And shouldn't be combined with unrelated version bumps. Smashing all together hurts transparency and traceability.

@webknjaz
Copy link
Member

webknjaz commented Feb 1, 2024

I merged dependabot's PR for cibuildwheel. MyPy bump should be separate, but there's no rush to do that — it doesn't seem related and I'm fine waiting for dependabot to send a PR, it's not a priority.

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Feb 1, 2024
CHANGES/937.bugfix.rst Outdated Show resolved Hide resolved
.pre-commit-config.yaml Outdated Show resolved Hide resolved
@Jamim Jamim changed the title Fix CI issues 🐛 Fix pytest/mypy compatibility issue for Python 3.7 Feb 1, 2024
CHANGES/937.bugfix.rst Outdated Show resolved Hide resolved
@Jamim Jamim force-pushed the fix/ci branch 2 times, most recently from f83450d to e85c449 Compare February 1, 2024 02:07
@Jamim Jamim requested a review from webknjaz February 1, 2024 02:13
@webknjaz webknjaz changed the title 🐛 Fix pytest/mypy compatibility issue for Python 3.7 🐛 Use compatible pytest < 8 when running MyPy against Python 3.7 Feb 1, 2024
@webknjaz webknjaz merged commit 770445a into aio-libs:master Feb 1, 2024
13 checks passed
@Jamim Jamim deleted the fix/ci branch February 1, 2024 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants