-
Notifications
You must be signed in to change notification settings - Fork 8
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
LOBSTER-Codebeamer tool to use configuration files #137
base: main
Are you sure you want to change the base?
Conversation
SurajBDeore
commented
Dec 3, 2024
- Replaced command-line arguments with a YAML-based configuration file for improved usability and maintainability.
- Simplifies parameter management and enhances readability.
c8021cc
to
824f7e5
Compare
- Replaced command-line arguments with a YAML-based configuration file for improved usability and maintainability. - Simplifies parameter management and enhances readability.
824f7e5
to
4d1e10c
Compare
ap.error("please set CB_ROOT or use --cb-root") | ||
sys.exit("lobster-codebeamer: Please set 'root' in the config file") |
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.
I observed that Argument parser error was replaced by sys.exit. Any specific reason for that?
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.
Because we are switching from command-line arguments to a config YAML file, it is essentially an argument parser error.
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 is a breaking change. Let's build a new release first, and merge afterwards.
using the --schema argument, or configure it in the config file. | ||
2. Configure the 'refs' upstream reference (optional) using --config or | ||
specify directly via command line. | ||
1. Specify the schema of the trace to be generated (optional, defaults to "Requirement"). You can set it using the |
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 point is difficult to understand