-
Notifications
You must be signed in to change notification settings - Fork 37
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
Documentation: Update Truth Data instructions #2154
Merged
Merged
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
bc773b8
updated contrib guide instructions to use GitHub Actions workflow to …
georgemccabe 0d19893
added step to confirm that new use case input data has been made avai…
georgemccabe 7ace292
added missing label for link
georgemccabe 4647b73
per PR #2154 review feedback, added info about the error that will oc…
georgemccabe a24aa16
added GitHub Action workflow to update input test data to use if work…
georgemccabe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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 | ||||
---|---|---|---|---|---|---|
|
@@ -882,6 +882,8 @@ chapter under | |||||
Pull Request Reviewer Instructions | ||||||
================================== | ||||||
|
||||||
.. _update-the-develop-data-directory: | ||||||
|
||||||
Update the develop data directory | ||||||
--------------------------------- | ||||||
|
||||||
|
@@ -1071,8 +1073,9 @@ then combine the two list items into a single item:: | |||||
|
||||||
- "met_tool_wrapper:0-54" | ||||||
|
||||||
Create a pull request from develop into develop-ref | ||||||
--------------------------------------------------- | ||||||
|
||||||
Update the Truth Data | ||||||
--------------------- | ||||||
|
||||||
The addition of a new use case results in new output data. When this happens, | ||||||
the reference branch needs to be updated so that future pull requests will | ||||||
|
@@ -1082,46 +1085,46 @@ the destination branch. This is done so that the pull request number | |||||
responsible for the changes in the truth data can be referenced to easily | ||||||
track where differences occurred. | ||||||
|
||||||
Merging develop into develop-ref often causes strange conflicts. It really is | ||||||
necessary and important to update develop-ref with the latest content of | ||||||
develop. Follow these command line instructions in the METplus repository to | ||||||
reconcile the conflicts before creating the pull request. | ||||||
|
||||||
* Reconcile conflicts between develop and develop-ref branches. | ||||||
|
||||||
:: | ||||||
|
||||||
git checkout develop-ref | ||||||
git pull | ||||||
git checkout develop | ||||||
git pull | ||||||
git merge -s ours develop-ref | ||||||
git push origin develop | ||||||
|
||||||
* Next click | ||||||
`here <https://github.com/dtcenter/METplus/compare/develop-ref...develop>`_ | ||||||
and click the green "Create pull request" button to create the pull request. | ||||||
|
||||||
.. figure:: figure/develop_into_develop-ref.png | ||||||
|
||||||
* Set the name of the pull request to "Update develop-ref after #XXXX" where | ||||||
XXXX is the pull request number that introduced the differences. | ||||||
|
||||||
* Delete the template content and add the pull request number (formatted #XXXX) | ||||||
and a brief description of what has changed. The description is optional | ||||||
because the link to the pull request should contain this information. | ||||||
|
||||||
A GitHub Action workflow is available to handle this step. | ||||||
|
||||||
* Ensure that the develop data directory has been updated to include all of the | ||||||
new input data. | ||||||
Check with the reviewers of recent pull requests that add a new use case to | ||||||
confirm that the steps under :ref:`update-the-develop-data-directory` have | ||||||
been completed. If this step has not been completed, then the new use case(s) | ||||||
will fail and the new output data will not be added to the truth data set. | ||||||
* Navigate to https://github.com/dtcenter/METplus/actions/workflows/update_truth.yml | ||||||
or from the METplus GitHub page, click on the Actions tab, | ||||||
then click on "Update Truth Data" under menu on the left. | ||||||
* Click on the "Run workflow" button on the right. | ||||||
* Click on the Branch pull down and select "develop" unless you are updating | ||||||
the truth data for a bugfix on a main_vX.Y branch. | ||||||
* Enter the pull request numbers that warranted the update. | ||||||
Include the '#' symbol before the number to create a link to the PR. | ||||||
PRs from a repository other than METplus should include | ||||||
the repository name before '#' symbol. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
* Enter a brief summary of the changes. | ||||||
Developers can navigate to the PRs for more information. | ||||||
|
||||||
.. figure:: figure/update_truth_data.png | ||||||
|
||||||
* Click the "Run workflow" button. | ||||||
* A new workflow run should appear at the top of the list and complete quickly. | ||||||
* Click on the "Pull Requests" tab. | ||||||
A new pull request should have been created with the information that | ||||||
was entered. Click on the new pull request. | ||||||
* Verify that the information in this pull request is correct. | ||||||
If the "develop" branch was selected in the "Run workflow" menu, | ||||||
then the pull request should show **develop-ref <- develop**. | ||||||
* Add the appropriate project and milestone values on the right hand side. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
* Create the pull request. | ||||||
|
||||||
* Squash and merge the pull request. It is not necessary to wait for the | ||||||
* Scroll to the bottom of the pull request and click "Squash and merge." | ||||||
* Click "Confirm squash and merge." It is not necessary to wait for the | ||||||
automation checks to complete for this step. | ||||||
|
||||||
* Monitor the Testing automation run for the develop-ref branch and ensure that | ||||||
all of the use cases run successfully and the final step named | ||||||
"Create Output Docker Data Volumes" completed successfully. | ||||||
|
||||||
|
||||||
Clean Up DTC Web Server | ||||||
----------------------- | ||||||
|
||||||
|
Binary file not shown.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
George, I realize that this section is about adding a new use case. But I do think its worth noting in some way that changes to existing use cases and bugfixes can also change the output. Feel free to use the wording below... or re-word it however you see fit.