Skip to content

Conversation

@davidhassell
Copy link
Collaborator

Fixes #896

Copy link
Member

@sadielbartholomew sadielbartholomew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent stuff. I am sure there are further good candidates for match/case across the codebase which we can convert when appropriate, but good to have one in already (makes us seem quite modern - not that I've seen any walrus operators anywhere in our code yet!). Please merge.

Comment on lines +962 to +963
match operator:
case "gt":
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful 🥹

@davidhassell davidhassell merged commit 88eb636 into NCAS-CMS:main Oct 14, 2025
1 of 3 checks passed
@davidhassell davidhassell deleted the python-3.10 branch October 14, 2025 07:35
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.

Remove support for Python 3.9 (EOL)

2 participants