Skip to content

Fix typo in the xarray_erddap#442

Merged
ocefpaf merged 1 commit intoioos:mainfrom
nishantharkut:fix/module-docstring-typo
Mar 18, 2026
Merged

Fix typo in the xarray_erddap#442
ocefpaf merged 1 commit intoioos:mainfrom
nishantharkut:fix/module-docstring-typo

Conversation

@nishantharkut
Copy link
Contributor

Just a typo fix in the xarray_erddap file regarding """ERDDAPyBackendEntrypoint."""

Copilot AI review requested due to automatic review settings March 17, 2026 19:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the erddapy/xarray_erddap.py module docstring to correct a typo in the referenced backend entrypoint name, aligning it with the actual ERDDAPyBackendEntrypoint class defined in the file.

Changes:

  • Fixes the misspelling ERDDAPyBackebdEntrypointERDDAPyBackendEntrypoint in the module docstring.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ocefpaf
Copy link
Member

ocefpaf commented Mar 17, 2026

Please turn copilot off.

@nishantharkut
Copy link
Contributor Author

the copilot reviewer is already turned off actually. I have double checked all the settings.

@ocefpaf
Copy link
Member

ocefpaf commented Mar 18, 2026

the copilot reviewer is already turned off actually. I have double checked all the settings.

It is still posting unnecessary noise to the PR.

Screenshot From 2026-03-18 08-09-26

@nishantharkut
Copy link
Contributor Author

the copilot reviewer is already turned off actually. I have double checked all the settings.

It is still posting unnecessary noise to the PR.

Screenshot From 2026-03-18 08-09-26

I found the global setting, sorry for this, I really didn't realise that it's from my side. it was automatically turned on since 2-3 months.

@ocefpaf ocefpaf merged commit 6ef214a into ioos:main Mar 18, 2026
13 of 16 checks passed
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