NOAA-GFDL / pace

Re-write of FV3GFS weather/climate model in Python
Apache License 2.0
12 stars 11 forks source link

Update Code Review Checklist #33

Open oelbert opened 10 months ago

oelbert commented 10 months ago

Pace PRs still imply you should check your code against the old DSL team's checklist, which is not accessible to anyone outside AI2. If we want to still use this as a guide for Pace code it should be somewhere accessible, maybe in the repo itself? We can (should) also edit it so it's in line with our current standards (and omits the OOP refactor section), or we should just remove that bullet point from the PR text completely if we don't want to use it