GEOS icon indicating copy to clipboard operation
GEOS copied to clipboard

Add xml formatting CI test

Open cssherman opened this issue 5 years ago • 4 comments

What is the requested feature? In develop, there is now a make target that will format all of the files in the src and examples folder: make geosx_format_all_xml_files . Randy suggested incorporating this into a CI test.

Note: in order to build the formatting tools, the version of python used to configure the code needs to have the virtualenv package installed (which is pretty standard but not part of core). Alternately, we can build the tools off of a separate python distribution by setting a cmake variable (PYTHON_POST_EXECUTABLE).

cssherman avatar Aug 13 '20 23:08 cssherman

this can be closed, right?

paveltomin avatar Sep 14 '23 23:09 paveltomin

@cssherman ?

paveltomin avatar Sep 20 '23 23:09 paveltomin

@TotoGaz can this be closed?

paveltomin avatar Nov 14 '23 19:11 paveltomin

@paveltomin - Unless I'm mistaken, we are validating the xml files against the schema, but are not checking xml formatting (indentation, etc.) as part of the CI

cssherman avatar Nov 15 '23 00:11 cssherman