Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add usage+help to all scripts #331
base: master
Are you sure you want to change the base?
Add usage+help to all scripts #331
Changes from all commits
9eaa9f5
165b4f8
6ba6ead
09f7b6a
60b594c
1ec6e90
842a969
ed77bda
0b48d10
2e69af6
563ddc4
bd70d13
8e4c59f
6af4dd8
d15e9ce
47783ff
cb357ca
92bda47
d38eb77
9bd19ce
ffc2193
e24499a
ba70c56
955f617
0f472bc
3b1285f
a67e19a
2f71372
c6f5bc1
43576b1
995f31b
fbcd436
9d8a9b2
efc133c
2ee3cbf
00fa80a
210795c
532aad8
7881226
e925b2d
a3722ac
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Please document at least the most important env vars and what happens if check fails (exit code=1?)
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.
Maybe add an example here on how to invoke this script (how to supply job id(s?)).
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.
Please be a bit more descriptive here.
Even reading the source code I'm still not sure what exactly this script does.
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.
Yep, me neither
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.
If nobody knows what this does, we might want to delete the script.
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.
No, the script is actively used within https://gitlab.suse.de/openqa/scripts-ci/
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.
Please include positional argument(s) in usage.