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.

Sign up for GitHub

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

Clean up code and add several minor features #750

Merged
merged 12 commits into from
Aug 15, 2022
Merged

Clean up code and add several minor features #750

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
58e3b8f
Update/improve debug_blocks output, see #718.
apcraig Aug 8, 2022
3b0d98d
Add ICE_MEMUSE cice.settings flag for batch memory use
apcraig Aug 8, 2022
6abb59b
Add setup_machparams.csh to compute batch/launch machine parameters
apcraig Aug 8, 2022
a14eb91
Update subroutine diagnostic_abort which calls print_state
apcraig Aug 8, 2022
f6d09e4
Update miniconda install information
apcraig Aug 8, 2022
a95c35c
Code cleanup based on compile with -Wall
apcraig Aug 9, 2022
d3b2294
Remove all trailing blank space with script
apcraig Aug 9, 2022
e630284
Update the cheyenne env so qc testing works
apcraig Aug 10, 2022
41b01f4
Update Icepack
apcraig Aug 12, 2022
ebb61a7
Clean up some output
apcraig Aug 12, 2022
8d08b7e
fix comments
apcraig Aug 14, 2022
bd8e1a8
update print_state output
apcraig Aug 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions doc/source/user_guide/ug_running.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -535,10 +535,10 @@ On macOS:

.. code-block:: bash

# Download the Miniconda installer to ~/Downloads/miniconda.sh
curl -L https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.sh -o ~/Downloads/miniconda.sh
# Download the Miniconda installer to ~/miniconda.sh
curl -L https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.sh -o ~/miniconda.sh
# Install Miniconda
bash ~/Downloads/miniconda.sh
bash ~/miniconda.sh
Comment on lines +538 to +541
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to take the time to fix this. If you update your commit message to "Closes #547" instead of "See #547", the issue will be closed automatically when your PR is merged :)

That could be done for the other issues you mention in previous commits, also, if you wish to close these issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to clarify, I added the "closes" to the PR documentation. I think this is a good place for it and how I usually do it. Interesting to know that similar syntax in the commit message could do the same thing.


# Follow the prompts

Expand Down