SciTools / cf-units

Units of measure as required by the Climate and Forecast (CF) Metadata Conventions
https://cf-units.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
64 stars 46 forks source link

Bump download/upload artifact gha version #396

Open bjlittle opened 9 months ago

bjlittle commented 9 months ago

🚀 Pull Request

Description

Require to bump the version of the download-artifact and upload-artifact to the latest version (@3 -> @4) consistently within the same pull-request.

bjlittle commented 9 months ago

Reference:

bjlittle commented 9 months ago

I think the best option here for cf-units is not to migrate to v4 for upload-artifact and consequently download-artifact due to the breaking change behaviour introduced.

I recommend that we sit tight and wait for the dust to settle, as community pressure may force a change or a documented best practice pattern may emerge for the common cf-units use case with cibuildwheels with a matrix of distros for wheels and sdist.