Skip to content

Conversation

@valeriupredoi
Copy link
Contributor

@valeriupredoi valeriupredoi commented Nov 21, 2022

Description

Unpin netcdf4 to free since we have repo patched iris with <1.6.1


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

@bouweandela bouweandela changed the title Unpin netcdf4 to free since we have repo patched iris with <1.6.1 Unpin NetCDF4 Nov 21, 2022
@bouweandela
Copy link
Member

Hi @valeriupredoi,

Looks good to me! I simplified the title of the pull request because it feels more readable to me (in the changelog) to have the explanation in the description rather than in the title. Is that OK with you?

@valeriupredoi
Copy link
Contributor Author

sure thanks @bouweandela 🍺 @ESMValGroup/technical-lead-development-team can one of you pls merge 🍻

@sloosvel sloosvel merged commit 3b02dde into main Nov 21, 2022
@sloosvel sloosvel deleted the unpin_netcdf4 branch November 21, 2022 16:25
@remi-kazeroni remi-kazeroni mentioned this pull request Nov 21, 2022
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🛑 DO NOT use netCDF4=1.6.1 or 1.6.2 and do not update your environment dynamically

4 participants