Skip to content

Rename Query Classes - #1465

Open
Sherwin-14 wants to merge 4 commits into
earthaccess-dev:mainfrom
Sherwin-14:rename-query-objects
Open

Rename Query Classes#1465
Sherwin-14 wants to merge 4 commits into
earthaccess-dev:mainfrom
Sherwin-14:rename-query-objects

Conversation

@Sherwin-14

@Sherwin-14 Sherwin-14 commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Description

Renamed the query classes so they don't read like results anymore. This one is a clean rename. If the consensus is to add deprecation path, I'm happy to add @deprecated aliases. Closes #1363.


AI usage disclosure

Using AI tools is welcome. Per our AI Policy,
we just ask that you disclose any use of generative AI tools in this contribution.

  • I have disclosed AI tool usage below, or confirmed that no AI tools were used

"Ready for review" checklist

  • Place this Pull Request (PR) in draft until it is ready for review (see below)
  • Please review our Pull Request Guide
  • Mark "ready for review" after following instructions in the guide

Merge checklist

  • PR title is descriptive
  • PR body contains links to related and resolved issues (e.g. closes #1)
  • If needed, CHANGELOG.md updated
  • If needed, docs and/or README.md updated
  • If needed, unit tests added (unsure how? see below!)
  • All checks passing (tip: comment pre-commit.ci autofix if pre-commit is failing)
  • At least one approval

Need help? We welcome contributions at every experience level. You don't have to
write tests alone — open your PR and ask for help. It's also fine to let GitHub run tests
for you, via Continuous Integration (CI),
instead of running them locally. If anything fails and you're not sure why, just
mention @earthaccess-dev/maintainers in a comment and we'll work with you!


📚 Documentation preview 📚: https://earthaccess--1465.org.readthedocs.build/en/1465/

@github-actions

Copy link
Copy Markdown

Binder 👈 Launch a binder notebook on this branch for commit f2c7fdc

I will automatically update this comment whenever this PR is modified

@Sherwin-14 Sherwin-14 changed the title Rename CMR query classes to avoid confusion with result classes Rename Query classes to avoid confusion with Result classes Sep 12, 2026
@Sherwin-14 Sherwin-14 changed the title Rename Query classes to avoid confusion with Result classes Rename Query classes Sep 12, 2026
@Sherwin-14 Sherwin-14 changed the title Rename Query classes Rename Query Classes Sep 13, 2026
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.

Rename DataGranules -> DataGranulesQuery (same for other query objects)

1 participant