Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ruff config flagged by repo review #8611

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

tqa236
Copy link
Contributor

@tqa236 tqa236 commented Jan 16, 2024

  • xref Address repo-review suggestions #8239, fixed PC191 and RF202 as shown here
  • Tests added
  • User visible changes (including notable bug fixes) are documented in whats-new.rst
  • New functions/methods are listed in api.rst

Copy link

welcome bot commented Jan 16, 2024

Thank you for opening this pull request! It may take us a few days to respond here, so thank you for being patient.
If you have questions, some answers may be found in our contributing guidelines.

@max-sixty max-sixty merged commit 33d51c8 into pydata:main Jan 16, 2024
26 checks passed
Copy link

welcome bot commented Jan 16, 2024

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again! celebration gif

@max-sixty
Copy link
Collaborator

Thanks @tqa236 !

@tqa236 tqa236 deleted the fix-ruff-config-repo-review branch January 16, 2024 17:33
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.

2 participants