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

Fix: find_filepaths non-parallel #22

Merged
merged 3 commits into from
Nov 24, 2023
Merged

Fix: find_filepaths non-parallel #22

merged 3 commits into from
Nov 24, 2023

Conversation

sphamba
Copy link
Collaborator

@sphamba sphamba commented Nov 23, 2023

Summary

  • fix utils/list.py flatten_list method
  • fix raised exception when filtering filepath of unknown product
  • add test case for flatten_list
  • add test for io/find.py find_filepaths method (using flatten_list)

@coveralls
Copy link

Coverage Status

coverage: 56.199% (+0.5%) from 55.696%
when pulling e002278 on EPFL-ENAC:fix-find-non-parallel
into bc11c99 on ghiggi:main.

@sphamba sphamba requested a review from ghiggi November 23, 2023 11:39
@ghiggi
Copy link
Owner

ghiggi commented Nov 23, 2023

Thanks for this. Merge this when you prefer.
Ping me when you update the CI ;)

@sphamba sphamba merged commit 8233266 into ghiggi:main Nov 24, 2023
14 of 15 checks passed
@sphamba sphamba deleted the fix-find-non-parallel branch November 24, 2023 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ghiggi ghiggi ghiggi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sphamba @coveralls @ghiggi