-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add documentation for extra job list #23827
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 documentation for extra job list #23827
Conversation
The new list was added in 0c53bb9.
tyler-yankee
left a comment
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.
+a:@mwoehlke-kitware for review, please. (I'm not sure whether we'll need platform here, @jwnimmer-tri?)
Reviewable status: LGTM missing from assignee mwoehlke-kitware, needs platform reviewer assigned, needs at least two assigned reviewers, missing label for release notes (waiting on @tyler-yankee)
mwoehlke-kitware
left a comment
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.
Reviewable status: needs platform reviewer assigned, needs at least two assigned reviewers, missing label for release notes (waiting on @tyler-yankee)
mwoehlke-kitware
left a comment
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.
@mwoehlke-kitware reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: needs platform reviewer assigned, needs at least two assigned reviewers, missing label for release notes (waiting on @tyler-yankee)
tyler-yankee
left a comment
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.
+a:@sammy-tri for platform review (per schedule), please.
Reviewable status: LGTM missing from assignee sammy-tri(platform), missing label for release notes (waiting on @tyler-yankee)
sammy-tri
left a comment
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 got way confused by this repo for a bit though)
Reviewable status:
complete! all discussions resolved, LGTM from assignees sammy-tri(platform),mwoehlke-kitware
sammy-tri
left a comment
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.
@sammy-tri reviewed 1 of 1 files at r1, all commit messages.
Reviewable status:complete! all discussions resolved, LGTM from assignees sammy-tri(platform),mwoehlke-kitware
3b2e9f9
into
RobotLocomotion:jenkins-jobs-experimental
The new list was added in 0c53bb9.
Towards #23799.
This change is