Ouranosinc / xclim / 18499870221 / 6
92%
main: 92%

DEFAULT BRANCH: main
Ran
Files 61
Run time 4s
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: 88.76% (-0.04%) from 88.796%
18499870221.6

push

github

web-flow 
Fix history handling (#2251)

<!--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:
- [x] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #2240 
- [x] Tests for the changes have been added (for bug fixes / features)
- [ ] (If applicable) Documentation has been added / updated (for bug
fixes / features)
- [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?

* When `as_dataset=True` is activated, "history" is not added on the
dataarray. Adding an history to a variable is not CF-compliant. We are
doing it because we wouldn't have this information otherwise. But when
in dataset mode, there's no need to do it (it is already added to the
dataset).
* The `update_history` function was adding the new history at the end of
the string. It should be at the top. The CF convention says "append",
the netCDF user guide says "append", but I'm pretty sure that's loose
language as all nc utilities I've encountered (`ncks` and the like)
update the history by _prepending_.

### Does this PR introduce a breaking change?
As in most cases, this is a gray area.

### Other information:

7557 of 8514 relevant lines covered (88.76%)

0.89 hits per line

Source Files on job run-3.11-ubuntu-latest-standard - 18499870221.6