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

Testscriptswolf #7

Merged
merged 4 commits into from
Jun 9, 2017
Merged

Conversation

eclare108213
Copy link
Contributor

There are two different things in this pull request, changes to the testing scripts for wolf and the change to the README.md file, which I pushed (on the wrong branch) by mistake. There's an artificial conflict in the README.md file (might be whitespace?) which shouldn't matter; I haven't figured out how to remove the commit or make the pull request only include the changes for wolf. Fixing this looks pretty complicated. So maybe just let it all go through together? And then maybe we should provide instructions on what NOT to do... (I'm learning)

@apcraig
Copy link
Contributor

apcraig commented Jun 9, 2017

So, I had to resolve the conflict before I could pull. I resolved it on the github page directly and did the commit there. It seems to have worked. Will now execute the PR.

@apcraig apcraig merged commit 9b8ece2 into CICE-Consortium:master Jun 9, 2017
@eclare108213 eclare108213 deleted the testscriptswolf branch June 9, 2017 00:20
JFLemieux73 pushed a commit to JFLemieux73/CICE that referenced this pull request Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants