-
Notifications
You must be signed in to change notification settings - Fork 37
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
The PR template seems a little outdated, so updating with some checklists of frequent items that come up during reviews.
- Loading branch information
Showing
1 changed file
with
14 additions
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,16 @@ | ||
<!-- -----------^ Click "Preview" for a functional view! --> | ||
## PR Checklists | ||
(Select the checklist that applies to your PR) | ||
|
||
## Why are you creating this Pull Request? | ||
Adding a Dataset Overview: | ||
- [ ] Dataset is available in the production VEDA STAC | ||
- [ ] Production STAC url is referenced in the dataset overview | ||
- [ ] All metadata, including those for the E&A page, is completed in the mdx file | ||
- [ ] Any images are compressed, ideally <500 KB | ||
- [ ] At least one code owner has reviewed the PR | ||
|
||
- [Adding Datasets or Stories](?title=Content%3A%20%3Cname%3E&expand=1&template=content.md) | ||
- [Version Release](?title=Deploy%20vX.X.X&expand=1&template=version_release.md) | ||
- [Other](?expand=1&template=default.md) | ||
Adding a Data Story: | ||
- [ ] Any data referenced in the story is already published to VEDA STAC and referenced in a separate dataset overview page (prior to merging or as part of the same PR) | ||
- [ ] All metadata is completed in the mdx file | ||
- [ ] Any images are compressed, ideally <500 KB | ||
- [ ] Published Date is as close to the actual release date as possible (work with veda-ui team to determine potential release date when story is close to ready) | ||
- [ ] At least one code owner has reviewed the PR |