-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove comment #2
base: auto-pr-comment-issue-824
Are you sure you want to change the base?
Remove comment #2
Conversation
ORNLDAAC no longer has any on-prem collections. This returns no collections: https://cmr.earthdata.nasa.gov/search/collections?data_center=ORNL_DAAC&cloud_hosted=false
- Use present tense (e.g., "add," not "added") - Order Remove section before Fixed section - Surround authors list with parentheses
8e22f11
to
1c589cd
Compare
1c589cd
to
60cd60e
Compare
ab78b00
to
a816c66
Compare
b626b77
to
7ab28f4
Compare
7ab28f4
to
971aad1
Compare
971aad1
to
b9e7fdd
Compare
b9e7fdd
to
0f6b2fa
Compare
0f6b2fa
to
b5e560e
Compare
b5e560e
to
e3da19a
Compare
Hello world! |
e3da19a
to
dd9a586
Compare
User chuckwondojo does not have permission to to run integration tests. A maintainer must perform a security review and re-run the failed jobs, if the code is deemed safe. |
dd9a586
to
8680ae7
Compare
User chuckwondojo does not have permission to run integration tests. A maintainer must perform a security review of the code changes in this pull request and re-run the failed integration tests jobs, if the code is deemed safe. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't work without the line that was removed.
User chuckwondojo does not have permission to run integration tests. A maintainer must perform a security review of the code changes in this pull request and re-run the failed integration tests jobs, if the code is deemed safe. |
b1dae63
to
9b447b3
Compare
User chuckwondojo does not have permission to run integration tests. A maintainer must perform a security review of the code changes in this pull request and re-run the failed integration tests jobs, if the code is deemed safe. |
2cd21a8
to
3ffbe2a
Compare
Pull Request (PR) draft checklist - click to expand
contributing documentation
before getting started.
title such as "Add testing details to the contributor section of the README".
Example PRs: #763
example
closes #1
. SeeGitHub docs - Linking a pull request to an issue.
CHANGELOG.md
with details about your change in a section titled## Unreleased
. If such a section does not exist, please create one. FollowCommon Changelog for your additions.
Example PRs: #763
README.md
with details of changes to theearthaccess 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 wewill help you out!
Request containing "pre-commit.ci autofix" to automate this.
📚 Documentation preview 📚: https://earthaccess--2.org.readthedocs.build/en/2/