fix test script #165 - Githubissues

c-blake / cligen

Nim library to infer/generate command-line-interfaces / option / argument parsing; Docs at

https://c-blake.github.io/cligen/

ISC License

508 stars 24 forks source link

fix test script #165 #166

Closed jiro4989 closed 4 years ago

jiro4989 commented 4 years ago

Overview

Version up 'setup-nim-action'
Skip test on 'macOS' VM because devel-compiler doesn't exist
Remove compiler-message from 'test/ref'
Change test script

Change test script

The current test script writes '--help' message and compiler message to 'test/out'. It is difficult to ignore with 'grep'. So I split 'compile' phase and 'run' command phase.

jiro4989 commented 4 years ago

wait, I will write description of this PR.

jiro4989 commented 4 years ago

I wrote description of PR.

jiro4989 commented 4 years ago

fmm, 'gh-pages' test failed.

c-blake commented 4 years ago

It is good to keep those compiler messages in test/ref (or test/out) for when I change something or Nim changes something to produce new warnings/errors. I'm not sure why the debian thing is spitting out "....." lines, but we could probably filter those out, too, like the grep -v '^CC'.

I'm not sure why I cannot silence that observable stores the ways I was trying. Silencing breaks for me locally, too, but works fine if I do an @if nimHasWarningObservable.. in my $HOME/.config/nim.cfg (that @if is actually used by compiler nim.cfg's, too).

jiro4989 commented 4 years ago

fmm... I understood.

c-blake commented 4 years ago

I merged this but then still had to fix some stuff to get it working. Basically, I had some things in my nim.cfg altering my output slightly. Now they are on the cmd line in test.sh. (I never really tried to match other-environment-build with my-environment-build before your CI stuff.)

TBH, I spent most of the time on this trying to get code-local warning deactivation of [ObservableStores] working. It still does not and I still don't know why. :(