Skip to content

Commit

Permalink
Update and rename PR-checklist.md to PR-review-checklist.md
Browse files Browse the repository at this point in the history
  • Loading branch information
anders-kolstad authored Dec 3, 2024
1 parent ae27cce commit 983bb9e
Showing 1 changed file with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
---
name: PR checklist
name: PR review checklist
about: A checklist for processing new PRs with indicator documentation
title: '[New PR]: "indicatorID"'
labels: PR-checklist
assignees: anders-kolstad

---

## PR checklist
## PR review checklist

This is a checklist for ecRxiv administrators that will have first contact with new PRs.
The list explains how to ensure the PR si following the standards for ecRxiv submissions,
Expand Down Expand Up @@ -101,7 +101,10 @@ Option 1 is the preferred option.

### When the file renders and the app don't break

Then we can merge the PR to main.
Add as many coomments and suggestions as you feel relevent in the PR.
You can also use the *community review* issue template for this.
When alle your comments are resolved by the author, then we can merge the PR to main.

Immediately afterwards we create a stable copy of the HTML with the version number
included in the name (INDICATORID_XXX_version_VVV_vvv; where XXX is the indicator ID number, and VVV and vv is the version number, before and after the perions, respectively).

Expand Down

0 comments on commit 983bb9e

Please sign in to comment.