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

Jump to bottom

Remove use of BATS in fms_io2 testing #358

Closed
edwardhartnett opened this issue Mar 25, 2020 · 1 comment · Fixed by #280
Closed

Remove use of BATS in fms_io2 testing #358

edwardhartnett opened this issue Mar 25, 2020 · 1 comment · Fixed by #280

Comments

@edwardhartnett
Copy link
Contributor

Some time ago, with the approval of @underwoo , I removed the use of the BATS utility in FMS testing. This was done because BATS is an unusual dependency, which caused a lot of configure problems for very little benefit. See #221

However, I notice bats is still being used in the fms_io2 directory.

Shall I remove bats from there as well?
@colingladueNOAA
Copy link
Contributor

I believe @wrongkindofdoctor is working on this in PR #280 .

@edwardhartnett edwardhartnett mentioned this issue Mar 25, 2020
Merged
8 tasks
This was linked to pull requests Mar 25, 2020
User/jml/update fms2io tests #280
Merged
User/jml/remove fms2io legacy f90 #290
Merged
@colingladueNOAA colingladueNOAA removed a link to a pull request Mar 25, 2020
User/jml/remove fms2io legacy f90 #290
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

User/jml/update fms2io tests
2 participants
@edwardhartnett @colingladueNOAA