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

Avoid considering non-default modular RPMs #1408

Closed
wants to merge 1 commit into from

Conversation

dralley
Copy link
Contributor

@dralley dralley commented Jul 17, 2019

Only modular artifacts that are "default" are permitted to be resolved
as dependencies of other modular or non-modular artifacts.

closes: #4962
https://pulp.plan.io/issues/4962

@dralley
Copy link
Contributor Author

dralley commented Jul 17, 2019

Unfinished

Blocked by: https://pulp.plan.io/issues/5151

@pep8speaks
Copy link

pep8speaks commented Jul 18, 2019

Hello @dralley! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-07-24 15:00:25 UTC

@dralley dralley force-pushed the mask-nonmodular-rpms branch 6 times, most recently from 84f11e7 to 26aee67 Compare July 24, 2019 14:50
Only modular artifacts that are "default" are permitted to be resolved
as dependencies of other modular or non-modular artifacts.

closes: #4962
https://pulp.plan.io/issues/4962
@dralley
Copy link
Contributor Author

dralley commented Sep 12, 2019

Closing for now -- May bring back if we deem necessary

@dralley dralley closed this Sep 12, 2019
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.

None yet

2 participants