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

Update usage of mypy.main in test following updates to mypy package #374

Conversation

rpmcginty
Copy link
Contributor

Last month, a mypy change was released that made all arguments for the mypy.main method keyword only. They also removed an unused argument.

python/mypy#13399

The alternative to this change is to freeze your version of mypy

Last month, a mypy change was released that made all arguments
for the mypy.main method keyword only.

python/mypy#13399

The alternative to this change is to freeze your version of mypy
@lidatong lidatong merged commit b76744b into lidatong:master Mar 18, 2023
@lidatong
Copy link
Owner

@rpmcginty thanks for this! mypy_main probably changed again probably between the time you submitted this PR and now... apologies again for that. i skipped the test to get checks passing on master, but feel free to remove that

i suppose we could pin mypy but to be honest i don't know enough about the typing ecosystem but i'm concerned about weird unlucky incompatibilities for users on different versions...

renovate bot referenced this pull request in ixm-one/pytest-cmake-presets Jun 12, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [dataclasses-json](https://togithub.com/lidatong/dataclasses-json) |
`0.5.7` -> `0.5.8` |
[![age](https://badges.renovateapi.com/packages/pypi/dataclasses-json/0.5.8/age-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://badges.renovateapi.com/packages/pypi/dataclasses-json/0.5.8/adoption-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://badges.renovateapi.com/packages/pypi/dataclasses-json/0.5.8/compatibility-slim/0.5.7)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://badges.renovateapi.com/packages/pypi/dataclasses-json/0.5.8/confidence-slim/0.5.7)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>lidatong/dataclasses-json</summary>

###
[`v0.5.8`](https://togithub.com/lidatong/dataclasses-json/releases/tag/v0.5.8)

[Compare
Source](https://togithub.com/lidatong/dataclasses-json/compare/v0.5.7...v0.5.8)

#### What's Changed

-
[#&#8203;341](https://togithub.com/lidatong/dataclasses-json/issues/341):
Fixes for type-parameters without arguments by
[@&#8203;USSX-Hares](https://togithub.com/USSX-Hares) in
[https://github.com/lidatong/dataclasses-json/pull/346](https://togithub.com/lidatong/dataclasses-json/pull/346)
- ci: disable 3.6 by [@&#8203;lidatong](https://togithub.com/lidatong)
in
[https://github.com/lidatong/dataclasses-json/pull/400](https://togithub.com/lidatong/dataclasses-json/pull/400)
- ci: bump gh action by
[@&#8203;lidatong](https://togithub.com/lidatong) in
[https://github.com/lidatong/dataclasses-json/pull/403](https://togithub.com/lidatong/dataclasses-json/pull/403)
- Update usage of mypy.main in test following updates to mypy package by
[@&#8203;rpmcginty](https://togithub.com/rpmcginty) in
[https://github.com/lidatong/dataclasses-json/pull/374](https://togithub.com/lidatong/dataclasses-json/pull/374)
- Fixes recursive dict encoding by
[@&#8203;matt035343](https://togithub.com/matt035343) in
[https://github.com/lidatong/dataclasses-json/pull/394](https://togithub.com/lidatong/dataclasses-json/pull/394)
- Removes unused type variables in api.py by
[@&#8203;matt035343](https://togithub.com/matt035343) in
[https://github.com/lidatong/dataclasses-json/pull/415](https://togithub.com/lidatong/dataclasses-json/pull/415)
- Documentation: added example of global_config by
[@&#8203;nr23](https://togithub.com/nr23) in
[https://github.com/lidatong/dataclasses-json/pull/189](https://togithub.com/lidatong/dataclasses-json/pull/189)
- ci: add publish to gh action workflow by
[@&#8203;lidatong](https://togithub.com/lidatong) in
[https://github.com/lidatong/dataclasses-json/pull/421](https://togithub.com/lidatong/dataclasses-json/pull/421)

#### New Contributors

- [@&#8203;rpmcginty](https://togithub.com/rpmcginty) made their first
contribution in
[https://github.com/lidatong/dataclasses-json/pull/374](https://togithub.com/lidatong/dataclasses-json/pull/374)
- [@&#8203;matt035343](https://togithub.com/matt035343) made their first
contribution in
[https://github.com/lidatong/dataclasses-json/pull/394](https://togithub.com/lidatong/dataclasses-json/pull/394)
- [@&#8203;nr23](https://togithub.com/nr23) made their first
contribution in
[https://github.com/lidatong/dataclasses-json/pull/189](https://togithub.com/lidatong/dataclasses-json/pull/189)

**Full Changelog**:
lidatong/dataclasses-json@v0.5.7...v0.5.8

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/ixm-one/pytest-cmake-presets).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTAuMCIsInVwZGF0ZWRJblZlciI6IjM1LjExMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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