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

Fsd doc #1

Merged
merged 12 commits into from
Oct 24, 2019
Merged

Fsd doc #1

merged 12 commits into from
Oct 24, 2019

Conversation

eclare108213
Copy link
Collaborator

Moving changes from FSD documentation branch into the fsd branch with the rest of the code changes, so that they can be combined into one PR for the Icepack master branch.

  • Developer(s):
    Lettie Roach

  • Please suggest code Pull Request reviewers in the column at right.

  • Are the code changes bit for bit, different at roundoff level, or more substantial?
    BFB

  • To verify that this PR passes the initial QC tests, please include the link to test results
    or paste in below the summary block from the bottom of the testing output.
    This will be tested as part of the PR into the master branch.

  • Does this PR create or have dependencies on CICE or any other models?
    No

  • Is the documentation being updated with this PR? (Y/N). Yes
    If not, does the documentation need to be updated separately at a later time? (Y/N)
    The documentation will need to be checked as part of the master PR review.

Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.

  • Other Relevant Details:
    I think this will work... should be easier than checking out the fsd branch and manually moving the changes! @lettie-roach if you'll review these PR changes and approve them, then either of us can do the merge.

@eclare108213 eclare108213 self-assigned this Oct 23, 2019
@eclare108213
Copy link
Collaborator Author

Just noticed a typo in sg_itd.rst: ice_ice should probably be ice_init.
This can be fixed this now, or later...

@lettie-roach lettie-roach merged commit 70f3442 into ifsd1 Oct 24, 2019
@lettie-roach lettie-roach deleted the fsd-doc branch October 24, 2019 13:40
lettie-roach pushed a commit that referenced this pull request Oct 18, 2022
* modifying PR template

* updating PR template

* adding link to documentation for PR (#1)

* modifying PR template

* typo

* modifying wording

* modifying link per PBsuggestion

* adding second link

* adding Tony's changes

* typo

* Making ECH suggestions
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.

2 participants