Skip to content

Add decision record doc for moving repo #1047

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mfisher87
Copy link
Collaborator

@mfisher87 mfisher87 commented Jul 8, 2025

Pull Request (PR) draft checklist - click to expand
  • Please review our
    contributing documentation
    before getting started.
  • Populate a descriptive title. For example, instead of "Updated README.md", use a
    title such as "Add testing details to the contributor section of the README".
    Example PRs: #763
  • Populate the body of the pull request with:
  • Update CHANGELOG.md with details about your change in a section titled
    ## Unreleased. If such a section does not exist, please create one. Follow
    Common Changelog for your additions.
    Example PRs: #763
  • Update the documentation and/or the README.md with details of changes to the
    earthaccess interface, if any. Consider new environment variables, function names,
    decorators, etc.

Click the "Ready for review" button at the bottom of the "Conversation" tab in GitHub
once these requirements are fulfilled. Don't worry if you see any test failures in
GitHub at this point!

Pull Request (PR) merge checklist - click to expand

Please do your best to complete these requirements! If you need help with any of these
requirements, you can ping the @nsidc/earthaccess-support team in a comment and we
will help you out!

  • Add unit tests for any new features.
  • Apply formatting and linting autofixes. You can add a GitHub comment in this Pull
    Request containing "pre-commit.ci autofix" to automate this.
  • Ensure all automated PR checks (seen at the bottom of the "conversation" tab) pass.
  • Get at least one approving review.

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

Copy link

github-actions bot commented Jul 8, 2025

Binder 👈 Launch a binder notebook on this branch for commit 160a6ac

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 3de24e9

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

Binder 👈 Launch a binder notebook on this branch for commit 653d8e5

@mfisher87 mfisher87 marked this pull request as draft July 8, 2025 18:31
Co-authored-by: Joseph H Kennedy <[email protected]>
Co-authored-by: danielfromearth <[email protected]>
Co-authored-by: Amy Steiker <[email protected]>
Co-authored-by: Jessica Scheick <[email protected]>
@mfisher87 mfisher87 force-pushed the move-repo-decision-record branch from 3de24e9 to aa28ee9 Compare July 8, 2025 18:58
@asteiker
Copy link
Member

@mfisher87 I know we started this up collaboratively at the hackday a few weeks ago and wanted to check in: Were you planning to continue work on this, and/or hoping others can contribute to move this out of a draft state? let me know how I can help support.

@mfisher87
Copy link
Collaborator Author

Glad you followed up, this fell off my radar. We need to convene some deciders and make the final review of this document and decide. I think we've recorded all of the alternatives.

@mfisher87
Copy link
Collaborator Author

pre-commit.ci autofix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants