Skip to content

Add functionality for multi IG environments#43

Open
florianschoffke wants to merge 14 commits into
mainfrom
feature/multi-ig
Open

Add functionality for multi IG environments#43
florianschoffke wants to merge 14 commits into
mainfrom
feature/multi-ig

Conversation

@florianschoffke
Copy link
Copy Markdown

Introduce support for multiple Implementation Guides (IGs) in the project. This includes enhancements to the command-line interface for selecting IGs, updates to the publishing and building processes, and the addition of a configuration file for multi-IG setups. Tests validate the new functionality and ensure compatibility with existing workflows.

@florianschoffke florianschoffke requested a review from cybernop May 22, 2026 05:21
@florianschoffke florianschoffke self-assigned this May 22, 2026
Copy link
Copy Markdown
Contributor

@cybernop cybernop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase your branch or merge the latest changes into your branch

Comment thread src/fhir_scripts/build.py Outdated
Comment on lines +24 to +39
def add_target_args(target_parser: ArgumentParser):
target_parser.add_argument(
"--ig",
action="extend",
nargs="+",
default=[],
help=(
"Target IG name(s), e.g. 'fhirscripts build pipeline --ig core rx' "
"or '--ig core --ig rx'"
),
)
target_parser.add_argument(
"--all",
action="store_true",
help="Run for all IGs, e.g. 'fhirscripts build pipeline --all'",
)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why don't you add those arguments to subparser or parser so it would allow using them for all sub command and instead add them them to each sub command individually.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ist angepasst

@florianschoffke florianschoffke requested a review from cybernop May 22, 2026 10:51
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