Ouranosinc / xscen / 25573300694 / 12
90%
main: 90%

DEFAULT BRANCH: main
Ran
Files 17
Run time 1s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 87.304% (-0.005%) from 87.309%
25573300694.12

push

github

web-flow 
Fix for change in parse 1.21 (#729)

<!-- Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [ ] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #xyz
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [x] (If applicable) Tests have been added.
- [x] This PR does not seem to break the templates.
- [x] CHANGELOG.rst has been updated (with summary of main changes).
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added.

### What kind of change does this PR introduce?
`parse` 1.21 has added support for the parsing of the grouping character
of decimal numbers. For example `1_000_000` uses `_`. The decoding of
this format spec modifier is done _before_ checking if the requested
format is an "extra" (custom) type. Thus, our custom `_` format was
misinterpreted and our custom regex never applied.

Result : the `{?:_}` pattern matched _anything_. It is supposed to match
everything except slashes `\ /`. This makes `parse_directory` fail to
properly match file names. See the modified test for an example.


I also added a few checks on the (rarely used) decorator that registers
these types.

### Does this PR introduce a breaking change?
No.


### Other information:
@aslibese FYI! I suggest you make sure you have this fix or `parse <
1.21` (1.20.2 works fine) in the environment that builds the catalog!

4291 of 4915 relevant lines covered (87.3%)

0.87 hits per line

Source Files on job run-Python3.12-pypi-macos-latest - 25573300694.12