Skip to content

Added: optional argument --infolder#17

Merged
filipesmg merged 5 commits into
FZJ-JSC:mainfrom
Matth-L:main
Jan 9, 2026
Merged

Added: optional argument --infolder#17
filipesmg merged 5 commits into
FZJ-JSC:mainfrom
Matth-L:main

Conversation

@Matth-L
Copy link
Copy Markdown
Contributor

@Matth-L Matth-L commented Jan 8, 2026

Hello,

I thought of an additional argument to be used: --infolder. Using this argument would allow the user to take all .json files from a folder and process them..

@filipesmg
Copy link
Copy Markdown
Collaborator

That's a good idea, I often had to use this. I think what I implemented was to give a list of json files only (like *.json). What do you think if we use the same existing way - that is, keeping:

parser.add_argument("file", nargs="+", help="File including list of running and recently-finished jobs or JSON file of a job")

But then we do a check like the one I implemented in the git.py plugin? We can give in a single or a list of json files, or a folder or list of folders (or a mix). I think that would be quite generic and simple to input.

@Matth-L
Copy link
Copy Markdown
Contributor Author

Matth-L commented Jan 8, 2026

I originally thought the same thing, but wanted to make minimal changes to how this script originally worked. I'm pushing the modification.

@filipesmg filipesmg merged commit 036c530 into FZJ-JSC:main Jan 9, 2026
1 check passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants