Gh318.wif

[WardF]

This is #318 with conflicts resolved. I expect the tests will initially fail, as they currently rely on v4.8.1 as well as v4.9.0, and the reliance of this code on v4.9.0 features is part of how it got so tangled up.

[edwardhartnett]

If you want this to work with versions of netcdf-c before 4.9.0, we can do that. We can even do it before the next netcdf-fortran release. Here's how I would approach that:
1 - merge this PR to get quantize in, turning off CI testing with any version other than 4.9.0.
2 - merge any other netcdf-fortran PRs that are desired.
3 - rename all files to F90 and F. This will cause the pre-procesor to be used.
4 - edit this code to guard it with pre-processor symbols.
5 - turn on CI testing for previous netcdf-c versions.

I'm happy to do this in the next few days. Let me know what you think...

[WardF]

@edwardhartnett That sounds good; getting the tests running to see what happens (not sure why they didn't run yesterday, probably the 'can't merge, there's a conflict'. The changes you outline would be great, and if you want to put up a PR it would be gratefully accepted. If not, I'll take a shot at it, although it will take a little bit of time. As it stands, just letting these tests run, wait for them to fail (as expected), make the change so that this then passes, and start merging the PR queue. Once that's done, we can rename to F90 and F, make the code changes, re-introduce previous versions of netCDF-C to the GitHub Actions.