Skip to content

default historical now True by default for get_job#74

Merged
pmrich merged 1 commit intomainfrom
historical-true-for-get-job
Apr 9, 2026
Merged

default historical now True by default for get_job#74
pmrich merged 1 commit intomainfrom
historical-true-for-get-job

Conversation

@bcote-anl
Copy link
Copy Markdown

No description provided.

@bcote-anl bcote-anl requested review from gabor-lbl and juztas April 9, 2026 15:27
Copy link
Copy Markdown
Contributor

@pmrich pmrich left a comment

Choose a reason for hiding this comment

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

Agree with this for get_job. Also agree that we should not do this for get_jobs.

Copy link
Copy Markdown
Contributor

@juztas juztas left a comment

Choose a reason for hiding this comment

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

This is fine by me!

@gabor-lbl
Copy link
Copy Markdown
Contributor

Looks good to me. Thanks!

@pmrich pmrich merged commit 392dcd2 into main Apr 9, 2026
1 check passed
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.

4 participants