Skip to content

Comments

Do not include mypy volume by default#25958

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-mypy-volume-for-regular-breeze-commands
Aug 26, 2022
Merged

Do not include mypy volume by default#25958
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-mypy-volume-for-regular-breeze-commands

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Aug 25, 2022

MyPy volume was included by default in shell command, but it is
not needed and might lead to missing mypy-cache volume problem.

It is useful for debugging mypy problems so it is still useful to
have it as an option of shell command.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@potiuk
Copy link
Member Author

potiuk commented Aug 25, 2022

cc: @ferruzzi

@ferruzzi
Copy link
Contributor

Thanks for this. 👍 (non-binding +1 ? :P )

@potiuk potiuk force-pushed the fix-mypy-volume-for-regular-breeze-commands branch from 37266a3 to fffff4f Compare August 25, 2022 17:30
@potiuk potiuk requested a review from mik-laj as a code owner August 25, 2022 17:30
@potiuk potiuk force-pushed the fix-mypy-volume-for-regular-breeze-commands branch from fffff4f to 1b57a0a Compare August 25, 2022 17:37
MyPy volume was included by default in shell command, but it is
not needed and might lead to missing mypy-cache volume problem.

It is useful for debugging mypy problems so it is still useful to
have it as an option of shell command.
@potiuk potiuk force-pushed the fix-mypy-volume-for-regular-breeze-commands branch from 1b57a0a to fd58b99 Compare August 25, 2022 17:41
@potiuk potiuk requested review from eladkal and ephraimbuddy August 25, 2022 18:01
@potiuk potiuk merged commit a8d9df1 into apache:main Aug 26, 2022
@potiuk potiuk deleted the fix-mypy-volume-for-regular-breeze-commands branch August 26, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants