Skip to content
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

Update develop-ref after #868 #873

Merged
merged 20 commits into from
Apr 12, 2021
Merged

Update develop-ref after #868 #873

merged 20 commits into from
Apr 12, 2021

Conversation

georgemccabe
Copy link
Collaborator

We discovered that files read from the file system in GitHub Actions were appearing in a different order than previous runs. I could not replicate this in Docker using the same container used for running the tests. The new order was causing differences and/or failures in some of the use case runs. We made minor updates to some of the use cases to prevent this from happening, such as changing a python embedding script that loops over observation files to process to sort the list first.

georgemccabe and others added 19 commits March 25, 2021 14:05
…on web server so develop-ref can check for new develop data
…ip instead of custom version, added info to contrib guide about CI
* Added section for ReadTheDocs, added basic info about the Release Guide and Verification Datasets Guide, but more information on those needs to be added later

* Added information about the various versions available in the documentation

* Updated the ReadTheDocs section with information on hidden branches and how to access them; Modified dtcenter.github.io references

* Added section to add_use_case.rst about Accessing the Documentation and modified documentation.rst to indicate the docs can still be built manually

* Modified format of list

* Modified formatting

* Changed GitHub Pages links to Read the Docs links

* Modified formatting

* Fixed typo

* Updated Release Guide for Read The Docs

* Took out duplicate data and consolidated sections

* Update docs/Contributors_Guide/add_use_case.rst

Removed sentence

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/add_use_case.rst

Fixed typo

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/add_use_case.rst

Removed end of previously removed sentence.

Co-authored-by: George McCabe <[email protected]>

Co-authored-by: George McCabe <[email protected]>
@georgemccabe georgemccabe added this to the METplus-4.0.0 milestone Apr 12, 2021
@georgemccabe georgemccabe merged commit 1b3776c into develop-ref Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants