Eccodes 2v36

[pp-mo]

Fixes for a couple of backwards-incompatibilities introduced in eccodes version 2.36

N.B. this also tested OK against 2.35, for which these lines are required.
However, it's not clear how old an eccodes this will support, since indicatorOfUnitOfTimeRange is only aliassed on loading -- not in saving, where setting it always+only uses indicatorOfUnitForForecastTime (the new form).

From their release page, v2.33 is when indicatorOfUnitForForecastTime was introduced, which we now write on saves.
Tested against eccodes v2.33 and v2.32 : confirmed the new code works with 2.33 but not 2.32.
As stated in whatsnew + updated requirements yml files.

N.B. the old name indicatorOfUnitOfTimeRange is still used in grib1 loading, because that still requires it
(!yuck!)

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.16%. Comparing base (6b97a7e) to head (69a8e90).
Report is 44 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #504      +/-   ##
==========================================
+ Coverage   89.13%   89.16%   +0.02%     
==========================================
  Files           8        8              
  Lines        2467     2473       +6     
  Branches      418      420       +2     
==========================================
+ Hits         2199     2205       +6     
  Misses        170      170              
  Partials       98       98              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pp-mo]

NOTE: merging this should unblock subsequent PRs.

• [CI Bot] environment lockfiles auto-update #499
• 👁 TEMPLATE DEVIATIONS: synced file(s) with SciTools/.github #500
• chore: update pre-commit hooks #501
• Bump scitools/workflows from 2024.06.5 to 2024.07.0 in the dependencies group #502

[trexfeathers]

Thanks @pp-mo!

I used this for reference during my review: v2.36.0 release notes