-
Notifications
You must be signed in to change notification settings - Fork 6
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
docs: adr-001 cli implementation #347
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, this makes sense to me. A couple questions around the config.
--- | ||
x-trestle-template-version: 0.0.1 | ||
title: Implement CLI Framework | ||
status: accespted |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
status: accespted | |
status: accepted |
markdown-path-catalog: markdown/catalogs | ||
markdown-path-compdef: markdown/component-definitions | ||
ssp-index-path: ssp-index.json | ||
profile-upstreams: [<url>] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would be to support the sync-upstreams
command, correct? I think we could collapse this to a single field given that trestle
would import the upstream artifacts into the correct location based on model type.
git-committer-email: [email protected] | ||
models: | ||
# we could allow for multiple or keep this as one | ||
- oscal-model: "ssp" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This configuration would be per model type so I think markdown-path
makes more sense here. As well as ssp-index
. My line of thinking is if you need to specify a model type on the field, it should go under the model specific configuration. WDYT?
Description
Adds ADR for proposed Click CLI and config file implementation.