ESCOMP / ESCOMP/CAM-SIMA

Need xmllint Github Action

Open
#188 0 comments 0 reactions 1 assignee Claimed by @nusbaume View on GitHub
enhancement Github action testing system
Dominant language
No language data
Stars
9
Forks
22
Avg merge
8d 5h
Merged PRs (30d)
4

Description

With the new multiple namelist definition file capability, there will soon be multiple namelist definition XML files, along with the CAMDEN registry XML file, that may not necessarily be checked during the current unit testing worfklow. Thus it could be beneficial to run `xmllint` on these files whenever they are modified (or a new one is introduced), similar to what is done when applying `pylint` to python files.

On top of that, having a first-step `xmllint` check before running more complicated software tests using Github Actions could help the CI system avoid any extra, wasted setup for tests that we know will just fail immediately due to an XML error.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.