Owen Voke

Results 192 comments of Owen Voke

For additional data, the `submission_client` field could also be added (which could be set to `eavesdrop.fm`). I guess that depends on whether you want that included, or whether other people...

In Pest, `beforeAll()` is called before the test class is instantiated. Usage of `$this` can't be used there. This error may also happen if the test function is defined as...

The documentation always shows the latest versions functionality, so it will show features from 3.x onwards. I think it would be nice to have versioning, but not sure if the...

I think I'd prefer this as an additional `--kit=` flag. That way it's more obvious. Feel free to PR this if you'd like to.

I think that `development` was specifically set as the default value (and we aren't really wanting to use `APP_ENV` as it means that dotenv will pull this out). However, this...

For reference, tests that interact with the framework should likely be in `Feature` (rather than `Unit`).

I'm more meaning that in a default Laravel Zero application, the `Unit` directory won't have access to command calls (or the container) as they require booting the framework (and the...

Yes, can replicate the issue with that repository on macOS. 👍🏻 I'll try and look into why it hangs.

For a Laravel application, that is the testcase class you are meant to use (for tests that require the framework). So everything is working as expected. If you extend the...

Hi @rynomars, apologies for the delay on reviewing this. I think this makes sense to do (I must have overlooked that `-` isn't allowed in the prefix). I'm assuming that,...