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

Add modularity support #4644

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

croissanne
Copy link
Member

@croissanne croissanne commented Mar 11, 2025

Adds enabled_modules as a list of strings with format "$name:$stream".

Alternatively we could switch to [{"name": …, "stream": …}], not sure what ppl prefer.

@croissanne croissanne force-pushed the dnf-modularity branch 2 times, most recently from 7c8a717 to f1ae4e7 Compare March 14, 2025 11:31
@croissanne croissanne changed the title dnf modularity wip Add modularity support Mar 14, 2025
@croissanne croissanne marked this pull request as ready for review March 14, 2025 11:32
@croissanne croissanne requested a review from a team as a code owner March 14, 2025 11:32
@croissanne croissanne requested review from mvo5, thozza and schuellerf and removed request for a team March 14, 2025 11:32
@supakeen
Copy link
Member

Adds enabled_modules as a list of strings with format "$name:$stream".

Alternatively we could switch to [{"name": …, "stream": …}], not sure what ppl prefer.

Without looking at the code (I'll do that in a bit) it might feel like $name:stream would also accept a /profile; that issue is less apparent with structured data? I don't know :)

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