Skip to content

Conversation

@ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Mar 5, 2024

This PR is a bit overwhelming and touches almost all the files but this is the best time to do this b/c we have only 1 real PR pending. Here we have:

  • all lints in ruff are enabled
  • many docstrings improvements
  • code should be all typed now
  • small performance fixes
  • some functions are now kwargs only and this may be seen as a breaking change, next version should reflect that

I'm planning on self-merging this to avoid conflicts with upcoming PRs.

@ocefpaf ocefpaf merged commit 5e45a0e into ioos:main Mar 5, 2024
@ocefpaf ocefpaf deleted the ruff_all branch March 5, 2024 16:03
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.

1 participant