Fork me on GitHub
#kaocha
<
2020-07-03
>
plexus08:07:24

@jatkin do you think there is an opportunity here to improve the documentation?

JAtkins13:07:50

Maybe just a note in the repl section that the above command works. That was the first place I looked when I was trying to figure out how to do this.

JAtkins13:07:10

I don't know enough about it to make a PR unfortunately. I was in the middle of adding one but then I realized I didn't know how to make it reload the config. The normal --watch mode will look at your config file, but AFAICT this solution doesn't.

plexus15:07:45

hmmm that's a good point, I think this should work, but we should come up with a better solution than this:

(kao-watch/run (assoc (kao-repl/config) :kaocha/cli-options {:config-file "tests.edn"}))

plexus15:07:07

or maybe not, looking at kaocha,repl/config it already adds (assoc-in [:kaocha/cli-options :config-file] config-file), which should be enough to make the watcher watch it...

plexus15:07:37

if that doesn't work then I would say that's a bug and you're welcome to file an issue

plexus15:07:20

never mind, I filed two issues

JAtkins15:07:24

Well, look what I get for assuming things 🙂 I just tried editing the test edn. Works fine. (I have the :watch? option in the file). I'll make a pr with a note in the docs if you want

plexus15:07:40

that would be great!

JAtkins15:07:56

This is with the simple

(kwatch/run (krpl/config))

plexus15:07:42

yeah, I think that's a good recommendation. Please use the aliases watch and config in the documentation though

plexus15:07:19

keep in mind that only the doc/*.md files that start with numbers (01_.. to 09_...) are hand-written, the rest is generated from Cucumber tests

plexus15:07:39

adding it to 07_watch should be good

JAtkins15:07:24

Ok, I'll go and swap out the change in the pr.

plexus15:07:17

merged! thank you for contributing to Kaocha!

JAtkins15:07:38

No problem. I love using it - this has been my dream workflow for a while now 🙂