Skip to content

Conversation

@ehogan
Copy link
Member

@ehogan ehogan commented Nov 20, 2025

Step 2 of completing #144.

PR creation checklist for the developer

  • Has <issue_number> above ☝️ been replaced with the issue number?
  • Has main been selected as the base branch?
  • Does the feature branch name follow the format <issue_number>_<short_description_of_feature>?
  • Does the text of the PR title exactly match with the text (not including the issue number) of the issue title?
  • Have appropriate reviewers been added to the PR (once it is ready for review)?
  • Has the PR been assigned to the developer(s)?
  • Have the same labels as on the issue (except for the good first issue label) been added to the PR?
  • Has the Climate Model Evaluation Workflow (CMEW) project been added to the PR?
  • Has the appropriate milestone been added to the PR?

Definition of Done for the developer

  • Does the change in this PR address the above issue / have all acceptance criteria been met?
  • Does the change in this PR follow the requirements in the wiki: Developer Guide (including copyrights)?
  • Have new tests related to the change been added?
  • Do all the GitHub workflow checks pass?
  • Do all the tests run locally and pass? (Note: the tests are not run by the GitHub workflow, see wiki: Run the tests locally)
  • Has the API documentation (e.g. docstrings in Python modules) related to the change been updated appropriately?
  • Has the user documentation (i.e. everything in the doc directory) related to the change been updated appropriately, including the Quick Start section?
  • Do the HTML pages render correctly? (See wiki: Build the documentation locally)

PR creation checklist for the reviewer

  • Has <issue_number> above ☝️ been replaced with the issue number?
  • Has main been selected as the base branch?
  • Does the feature branch name follow the format <issue_number>_<short_description_of_feature>?
  • Does the text of the PR title exactly match with the text (not including the issue number) of the issue title?
  • Have appropriate reviewers been added to the PR (once it is ready for review)?
  • Has the PR been assigned to the developer(s)?
  • Have the same labels as on the issue (except for the good first issue label) been added to the PR?
  • Has the Climate Model Evaluation Workflow (CMEW) project been added to the PR?
  • Has the appropriate milestone been added to the PR?

Definition of Done for the reviewer

  • Does the change in this PR address the above issue / have all acceptance criteria been met?
  • Does the change in this PR follow the requirements in the wiki: Developer Guide (including copyrights)?
  • Have new tests related to the change been added?
  • Do all the GitHub workflow checks pass?
  • Do all the tests run locally and pass? (Note: the tests are not run by the GitHub workflow, see wiki: Run the tests locally)
  • Has the API documentation (e.g. docstrings in Python modules) related to the change been updated appropriately?
  • Has the user documentation (i.e. everything in the doc directory) related to the change been updated appropriately, including the Quick Start section?
  • Do the HTML pages render correctly? (See wiki: Build the documentation locally)

@ehogan ehogan self-assigned this Nov 20, 2025
@ehogan ehogan added enhancement New feature or request configure Anything related to configuration recipe Anything related to ESMValTool labels Nov 20, 2025
@ehogan ehogan added this to the v0.1.0 (single recipe) milestone Nov 20, 2025
@ehogan
Copy link
Member Author

ehogan commented Nov 20, 2025

I missed a file move in #275. I will move doc/source/assessment_areas.rst to doc/source/recipe.rst in a separate PR after this one.

@ehogan
Copy link
Member Author

ehogan commented Nov 20, 2025

@alistairsellar apparently there are issues with the MASS tape library hardware right now, so CMEW doesn't complete because the standardise step is hanging. The unit tests pass though 😊

@ehogan ehogan merged commit 7791e66 into main Dec 4, 2025
3 checks passed
@ehogan ehogan deleted the 144_update_process_to_recipe_part_2 branch December 4, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

configure Anything related to configuration enhancement New feature or request recipe Anything related to ESMValTool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants