Update foreman utils.py: use list comprehension for collecting hung/lost jobs - Githubissues

AlexsLemonade / refinebio

Refine.bio harmonizes petabytes of publicly available biological data into ready-to-use datasets for cancer researchers and AI/ML scientists.

https://www.refine.bio/

Other

126 stars 19 forks source link

Update foreman utils.py: use list comprehension for collecting hung/lost jobs #3369

Closed tosemml closed 10 months ago

tosemml commented 10 months ago

Issue Number

Not an issue

Purpose/Implementation Notes

These slight code adjustments aim to enhance Pythonic quality by incorporating the list comprehension

Types of changes

What types of changes does your code introduce?

Code Refactoring

Checklist

Put an x in the boxes that apply.

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

arkid15r commented 10 months ago

Hi @tosemml, thank you for your contribution!

FYI: I've ran pre-commit to fix the code style and will be merging this PR soon.

tosemml commented 10 months ago

@arkid15r thanks a lot.