Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

various fixes, see commit messages AND descriptions #14

Merged
merged 8 commits into from
May 1, 2023
Merged

various fixes, see commit messages AND descriptions #14

merged 8 commits into from
May 1, 2023

Conversation

xHeaven
Copy link
Member

@xHeaven xHeaven commented May 1, 2023

No description provided.

xHeaven added 8 commits May 1, 2023 20:34
You don't need to specify `./vendor/bin/`, composer is smart enough to find the binaries. Also, removing the directories makes the scripts usable on Windows too.
Also, added `@` before `php` in the `phpunit-coverage` script. This makes sure that the PHP version to run the coverage test is the same as composer is currently running on.
- anonymous methods have been made `static`
- `__toString()` replaced with `(string)` cast
- removed unnecessary string casting
Variable functions are way faster than using call_user_func.
@CommandString CommandString merged commit 7f6c5c2 into tnapf:full-coverage May 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants