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
Don't complain about unowned string imports #14179
Don't complain about unowned string imports #14179
Conversation
src/python/pants/backend/python/dependency_inference/scripts/import_parser.py
Show resolved
Hide resolved
src/python/pants/backend/python/dependency_inference/parse_python_imports.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, thanks!
src/python/pants/backend/python/dependency_inference/parse_python_imports.py
Outdated
Show resolved
Hide resolved
src/python/pants/backend/python/dependency_inference/parse_python_imports.py
Outdated
Show resolved
Hide resolved
src/python/pants/backend/python/dependency_inference/parse_python_imports_test.py
Outdated
Show resolved
Hide resolved
src/python/pants/backend/python/dependency_inference/scripts/import_parser.py
Outdated
Show resolved
Hide resolved
src/python/pants/backend/python/dependency_inference/scripts/import_parser.py
Show resolved
Hide resolved
…hon_imports.py Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Updated with "strong"/"weak". Only possible improvements would be to maybe default in the tests, but I don't think I care that strongly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Sounds good to use weak
for now and not use the enum. Python enums are a lot of boilerplate :)
Ready for me to merge after green CI? |
🚀 |
The title says it all!