Skip to content

Add mypy-plugin extra dependency #1605

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

Merged
merged 2 commits into from
Jun 27, 2023

Conversation

zhukovgreen
Copy link
Contributor

@zhukovgreen zhukovgreen commented Apr 26, 2023

use pip install classes[compatible-mypy] for installing additional dependencies required by the mypy-plugin.

I have made things!

Checklist

  • I have double checked that there are no unrelated changes in this pull request (old patches, accidental config files, etc)
  • I have created at least one test case for the changes I have made
  • I have updated the documentation for the changes I have made
  • I have added my changes to the CHANGELOG.md

Related issues

🙏 Please, if you or your company finds dry-python valuable, help us sustain the project by sponsoring it transparently on https://github.com/sponsors/dry-python. As a thank you, your profile/company logo will be added to our main README which receives hundreds of unique visitors per day.

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zhukovgreen zhukovgreen force-pushed the add-mypy-plugin-extras branch from 22bf87d to f4d0905 Compare April 26, 2023 16:47
@zhukovgreen
Copy link
Contributor Author

Should be fine now

@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Merging #1605 (ac97a89) into master (c4643b0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            master     #1605    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           79        79            
  Lines         2494      2494            
  Branches       434       158   -276     
==========================================
  Hits          2494      2494            

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@zhukovgreen
Copy link
Contributor Author

Need some help here with the failure on py37. Vry probable that this failure is not related to the merge request

@sobolevn sobolevn closed this May 3, 2023
@sobolevn sobolevn reopened this May 3, 2023
pyproject.toml Outdated
@@ -48,6 +48,7 @@ _ = "returns.contrib.hypothesis._entrypoint"
python = "^3.7"

typing-extensions = ">=4.0,<5.0"
mypy = { version = "^1.2", optional = true }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can now remove mypy from dev-dependencies

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you think so? In this case, poetry install won't install mypy.

Copy link
Contributor Author

@zhukovgreen zhukovgreen Jun 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we remove the mypy from the dev-dependencies, then in order to let the mypy installed, we need to do:

$ poetry install --extras compatible-mypy

Otherwise mypy won't be installed. If we have the mypy of the same version in the dev dependencies, then poetry install will install it (because dev deps are included when poetry install).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zhukovgreen zhukovgreen force-pushed the add-mypy-plugin-extras branch from f4d0905 to 1c77547 Compare May 3, 2023 18:03
@zhukovgreen zhukovgreen force-pushed the add-mypy-plugin-extras branch 4 times, most recently from e4881d1 to ca4548b Compare June 26, 2023 15:23
Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, update mypy to ^1.4.0 and I will be happy to merge it!

@zhukovgreen zhukovgreen force-pushed the add-mypy-plugin-extras branch 2 times, most recently from 1ec2722 to df7170a Compare June 27, 2023 13:54
use `pip install classes[mypy-plugin]` for installin additional
dependencies required by the mypy-plugin.
Add mypy-plugin extra dependency

use `pip install classes[mypy-plugin]` for installing additional
dependencies required by the mypy-plugin.
@zhukovgreen zhukovgreen force-pushed the add-mypy-plugin-extras branch from df7170a to 3b905ef Compare June 27, 2023 13:58
@zhukovgreen
Copy link
Contributor Author

@sobolevn seems like all done now

@sobolevn sobolevn merged commit 2717d1f into dry-python:master Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants