Skip to content

Add option to run executable with a different group#498

Merged
sb10 merged 5 commits intodevelopfrom
set-exe-group
Jan 22, 2026
Merged

Add option to run executable with a different group#498
sb10 merged 5 commits intodevelopfrom
set-exe-group

Conversation

@mjkw31
Copy link
Collaborator

@mjkw31 mjkw31 commented Jan 22, 2026

No description provided.

@mjkw31 mjkw31 requested a review from sb10 January 22, 2026 15:35
Copy link
Member

@sb10 sb10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs rebase and more delinting. Also, I always feel like there are lots more places that these options have to be added, where similar types get created from other types, and there's usually bugs related to an option not being passed through in certain scenarios. Have a good check to see what happens with other options.

@mjkw31
Copy link
Collaborator Author

mjkw31 commented Jan 22, 2026

Also, I always feel like there are lots more places that these options have to be added, where similar types get created from other types, and there's usually bugs related to an option not being passed through in certain scenarios. Have a good check to see what happens with other options.

I chased one of the other fields through (ReqGroup) and saw where the job was cloned and got data from; I did that until I got to the command package.

@sb10 sb10 merged commit 32f385f into develop Jan 22, 2026
2 checks passed
@sb10 sb10 deleted the set-exe-group branch January 22, 2026 16:24
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