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

Update compat error checking to disallow "minimal" in concat() #9525

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

VeckoTheGecko
Copy link
Contributor

@VeckoTheGecko VeckoTheGecko commented Sep 20, 2024

  • Closes None (no related issue)
  • Tests added
  • User visible changes (including notable bug fixes) are documented in whats-new.rst
    • Not notable

Before:
AttributeError: 'IndexVariable' object has no attribute 'minimal'

After:
ValueError: compat='minimal' invalid: must be 'broadcast_equals', 'equals', 'identical', 'no_conflicts' or "override"

Copy link

welcome bot commented Sep 20, 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
Copy link
Collaborator

Excellent, thank you @VeckoTheGecko !

@max-sixty max-sixty enabled auto-merge (squash) September 20, 2024 17:18
@max-sixty max-sixty merged commit 2b800ba into pydata:main Sep 20, 2024
28 checks passed
Copy link

welcome bot commented Sep 20, 2024

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

@VeckoTheGecko
Copy link
Contributor Author

🥳

@VeckoTheGecko VeckoTheGecko deleted the compat branch September 20, 2024 17:36
hollymandel pushed a commit to hollymandel/xarray that referenced this pull request Sep 23, 2024
…ydata#9525)

* update compat error checking

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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