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.

Sign up for GitHub

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

Jump to bottom

Change id field passed to JobInfo from int to str #30

Merged
merged 2 commits into from
Jul 22, 2024
Merged

Change id field passed to JobInfo from int to str #30

merged 2 commits into from
Jul 22, 2024

Commits on Jul 20, 2024

  1. Change id field passed to JobInfo from int to str 

    fixes aiidateam#29

The id field of JobInfo is expecting a str. In aiidateam#24 when parsing JSON
output of `hq job list` the json loads will use the int for the id
parsed directly. Wrong type causes the subtle issue that when job is
waiting it not get into QUEUED state, but immediatly finished and get
nothing to parse from output.
    @unkcpz
    unkcpz committed Jul 20, 2024
    Configuration menu
    Copy the full SHA
    d590c45 View commit details
    Browse the repository at this point in the history

  2. [pre-commit.ci] auto fixes from pre-commit.com hooks 

    for more information, see https://pre-commit.ci
    @pre-commit-ci
    pre-commit-ci[bot] committed Jul 20, 2024
    Configuration menu
    Copy the full SHA
    ca89635 View commit details
    Browse the repository at this point in the history