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

Added support for maxRC defined in property file or with CLI option #109

Merged
merged 1 commit into from
May 30, 2022

Conversation

M-DLB
Copy link
Collaborator

@M-DLB M-DLB commented May 20, 2022

Added the possibility to specify a maximum acceptable return code, which can be specified either in the property file, or through the CLI option (which takes precedence). If none of these options is used, a default acceptable RC is set to 3 (which is the highest value considered as acceptable by IDz v15 Code Review feature).

@M-DLB
Copy link
Collaborator Author

M-DLB commented May 20, 2022

@dennis-behm Can you have a look please?
Thank you!

@dennis-behm
Copy link
Member

@M-DLB Code looks fine. Can you please update the Command Line Options Summary in the README.md with this new option?

@M-DLB
Copy link
Collaborator Author

M-DLB commented May 30, 2022

@dennis-behm I think it was already changed accordingly in this file https://github.com/IBM/dbb/pull/109/files#diff-6a506dec2a246c4779c878e043aade4ba820cd70b07cbaacaeeb32d87bf46b80
Unless it has to be changed elsewhere too?

@dennis-behm
Copy link
Member

Sorry, I must have looked at your main branch ...

@dennis-behm dennis-behm merged commit aae5e4d into IBM:main May 30, 2022
johnmnemec pushed a commit to johnmnemec/dbb that referenced this pull request Sep 6, 2022
…r with CLI option (IBM#109)

Signed-off-by: John Nemec <john.nemec@ibm.com>
johnmnemec pushed a commit to johnmnemec/dbb that referenced this pull request Sep 6, 2022
…r with CLI option (IBM#109)

Signed-off-by: John Nemec <john.nemec@ibm.com>
johnmnemec pushed a commit to johnmnemec/dbb that referenced this pull request Sep 6, 2022
johnmnemec pushed a commit to johnmnemec/dbb that referenced this pull request Sep 9, 2022
…r with CLI option (IBM#109)

Signed-off-by: John Nemec <john.nemec@ibm.com>
@M-DLB M-DLB deleted the RunCodeReview-MaxRC-Fix branch April 3, 2024 12:32
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