Skip to content

feat: per module requirements #749

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.

Sign up for GitHub

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

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

feat: per module requirements #749

wants to merge 6 commits into from

Conversation

qz267
Copy link
Contributor

@qz267 qz267 commented Jul 14, 2025 •
edited
Loading

Introduced:

  • per_module_roles: to config required roles to run each sub modules.
  • per_module_service: to config required services to run each sub modules.

Added feature flag --per-module-requirements to generate_docs cmd, to align with the per module configs.

@qz267 qz267 requested review from ayushmjain, imrannayer, q2w and a team as code owners July 14, 2025 20:29
@qz267 qz267 changed the title [WIP]feat: per module requirements feat: per module requirements Jul 15, 2025
q2w
q2w reviewed Jul 22, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@q2w q2w q2w left review comments

@pawan1210 pawan1210 pawan1210 approved these changes

@ayushmjain ayushmjain Awaiting requested review from ayushmjain ayushmjain is a code owner

@imrannayer imrannayer Awaiting requested review from imrannayer imrannayer is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@qz267 @q2w @pawan1210