hotfix/general.yaml again

esm-tools / esm_master

GNU General Public License v2.0

2 stars 2 forks source link

hotfix/general.yaml again #60

Closed mandresm closed 3 years ago

mandresm commented 3 years ago

really dumb bug, indentation missing...

mandresm commented 3 years ago

It would work if general.yaml existed, but there are some development branches that don't have it yet and there it breaks. This fix makes it general, as it should have been on the first place.

pgierz commented 3 years ago

@mandresm: if we really go for automatic tests, we can include one test that does stuff like syntax checks and style checks. I can make a preliminary PR for that if you want (that is more fun than my paper reviews)

mandresm commented 3 years ago

That would be a nice addition to the checks. By the way, I have esm_tests working quite nicely and now getting to populate it with runscripts. And yes, programming that kind of stuff is not only useful but real fun!