Ouranosinc / xscen

A climate change scenario-building analysis framework.
https://xscen.readthedocs.io/
Apache License 2.0
18 stars 2 forks source link

Skip "root" dirs too #479

Closed aulemahal closed 1 month ago

aulemahal commented 1 month ago

Pull Request Checklist:

What kind of change does this PR introduce?

Small fix upon #478. Skips even if a directory to skip is one of the entries in directory or a parent of one of the entries. I implemented it because I misunderstood an error I was having, but still, this could be used in the specific scenario where you are running parse_directory through an incremental stack of xscen configurations. To remove en entry of the directories list of a previous configuration, you would only need to add a skip_dirs entry, and not rewrite the filtered directories.

This also adds logging calls when skipping. And a precision to the doc about the absoluteness of paths to skip.

Does this PR introduce a breaking change?

No.

Other information: