Skip to content

Conversation

@pp-mo
Copy link
Owner

@pp-mo pp-mo commented Aug 14, 2025

Replaces #145
Removed the "pre-commit commit" from valeriupredoi:add_warning_and_tests and re-merged from main instead

Just because #145 diff was polluted with a lot of pre-commit changes.

valeriupredoi and others added 30 commits August 8, 2025 15:21
Co-authored-by: Patrick Peglar <patrick.peglar@metoffice.gov.uk>
Co-authored-by: Patrick Peglar <patrick.peglar@metoffice.gov.uk>
Co-authored-by: Patrick Peglar <patrick.peglar@metoffice.gov.uk>
Co-authored-by: Patrick Peglar <patrick.peglar@metoffice.gov.uk>
Co-authored-by: Patrick Peglar <patrick.peglar@metoffice.gov.uk>
@pp-mo pp-mo merged commit fbd2e93 into main Aug 14, 2025
5 checks passed
@pp-mo
Copy link
Owner Author

pp-mo commented Aug 14, 2025

@valeriupredoi many thanks for this
And for all your patience !!

Please let me know if you have other needs.
I'm hoping to get a new stable version out by the end of this month, as @schlunma suggested

@valeriupredoi
Copy link
Contributor

brilliant, many thanks to you, Patrick! Let me know if I can help in any other way - with packaging etc; we'll get back to you with more stuff as we now start using ncdata in anger, so expect me bugging you again in the future 😄 🍻

@pp-mo
Copy link
Owner Author

pp-mo commented Aug 14, 2025

brilliant, many thanks to you, Patrick! Let me know if I can help in any other way - with packaging etc; we'll get back to you with more stuff as we now start using ncdata in anger, so expect me bugging you again in the future 😄 🍻

😨 !

@valeriupredoi
Copy link
Contributor

brilliant, many thanks to you, Patrick! Let me know if I can help in any other way - with packaging etc; we'll get back to you with more stuff as we now start using ncdata in anger, so expect me bugging you again in the future 😄 🍻

😨 !

wile-e-coyote_1261

@pp-mo pp-mo deleted the pp__warning_and_tests branch September 9, 2025 13:38
@scitools-ci scitools-ci bot removed this from 🚴 Peloton Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants