search

florianschanda / miss_hit

MATLAB Independent, Small & Safe, High Integrity Tools - code formatter and more
GNU General Public License v3.0
158 stars 21 forks source link

Not add white spaces in tests for octave %! #261

Closed loisspitz closed 1 year ago

loisspitz commented 1 year ago

For built in test in Octave starting with %!test at the end of a file. Miss_hit does add an whitespace which beaks the tests. Implemented like the detection of pragma only executed when octave. The behavior is only changed when the language is set to to octave. Therefore a new parameter language.infile_tests is introduced. I hope this follows the intensional structure of miss_hit.

florianschanda commented 1 year ago

Hi, thank you very much for this idea. Indeed, it very much is in the spirit of the current structure of MH.

I will not merge this PR directly but push a similar but slightly wider implementation of this that also recognises the Octave .tst files as proper inputs.

florianschanda commented 1 year ago

I have added an acknowledgement in the CHANGELOG.md file, since I did not merge this PR directly.