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

Remove unused testmods directories #84

Merged
merged 1 commit into from
Oct 17, 2018

Conversation

billsacks
Copy link
Member

Just a bit of cleanup: As far as I can tell, these testmods directories are unused.

User interface changes?: No

Fixes: None

Testing: None

Copy link
Contributor

@fischer-ncar fischer-ncar left a comment

Choose a reason for hiding this comment

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

cesm2dev01 is mentioned in the documentation and should be changed to defaultio.

cime/doc/source/users_guide/testing.rst

Copy link
Contributor

@fischer-ncar fischer-ncar left a comment

Choose a reason for hiding this comment

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

I also wanted to make sure that @jedwards4b doesn't need the trilinos testmods.

@jedwards4b
Copy link
Contributor

I do not.

@billsacks
Copy link
Member Author

@fischer-ncar - thanks - see ESMCI/cime#2862

@fischer-ncar fischer-ncar merged commit aa79e92 into ESCOMP:master Oct 17, 2018
@fischer-ncar
Copy link
Contributor

@billsacks do you want to remove these testsmods from cesm2.1-alphabranch too?

@billsacks
Copy link
Member Author

@fischer-ncar thanks for checking. I figured it was more important to remove these from master, and I didn't take the time to look into whether they were used at all on cesm2.1-alphabranch. I'd be happy with their removal there, too, if they are unused there as well, if you have time to check if that's okay.

@billsacks billsacks deleted the remove_unused_testmods branch October 17, 2018 21:23
fischer-ncar added a commit to ESMCI/cime that referenced this pull request Oct 18, 2018
Remove reference to deleted testmods directories
Companion to ESCOMP/CESM#84

Test suite: none
Test baseline: n/a
Test namelist changes: n/a
Test status: bit for bit

Fixes none

User interface changes?: N

Update gh-pages html (Y/N)?: Y

Code review:
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.

None yet

3 participants