Ziinc
Ziinc
I've reproduced it here: https://github.com/Ziinc/mix-test-watch-repro Thanks for the super prompt reply!
Seems like the `command` variable containing build tasks cmds isn't passed to the windows `cmd` command https://github.com/lpil/mix-test.watch/blob/6dc3973076e82a2735d084387c351d25b098a6cb/lib/mix_test_watch/port_runner/port_runner.ex#L16 as compared to the linux version: https://github.com/lpil/mix-test.watch/blob/6dc3973076e82a2735d084387c351d25b098a6cb/lib/mix_test_watch/port_runner/port_runner.ex#L19
Made: https://github.com/lpil/mix-test.watch/pull/104 Isn't completely fixed though, there's a weird double quote issue that I explained in the PR.
@KineticFox realtime logs has recently been made available, you will be able to view your logs at this link, you can see if your client is connecting correctly to the...
@saltcod how did you get the second pic? It is missing a line in the error message, and the "No results" message actually gets hidden when there is an error.
@saltcod I'm open to merging the two types of alerts, I personally prefer the design/layout of the no results message. I'll defer to the design team to make the choice....
internal ticket [created](https://www.notion.so/supabase/Logs-Merge-No-Results-Alert-with-Error-Alerts-531c4b7aca87499b886d34c63aa79e65)
Splitting into smaller PRs
Thanks for contributions, will take over these changes from here. Closing for now due to conflicts.
@gitstart thanks for the contribution, I will take over this PR from here so that we can ship these changes faster.