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

Fix module_mapping to work regardless of capitalization and - vs …_ (Cherry-pick of #12068) #12070

Merged
merged 1 commit into from
May 13, 2021

Conversation

Eric-Arellano
Copy link
Contributor

[ci skip-rust]
[ci skip-build-wheels]

…`_` (pantsbuild#12068)

These all resolve to the same project:

- my-req
- my_req
- MY_REQ

Our `module_mapping` should not care about which format the original requirement string used.

This also adds `apache-airflow` to our default mapping.

[ci skip-rust]
[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano requested a review from tdyas May 13, 2021 01:35
@Eric-Arellano Eric-Arellano merged commit 32a1a09 into pantsbuild:2.5.x May 13, 2021
@Eric-Arellano Eric-Arellano deleted the cp-module-mapping branch May 13, 2021 02:05
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