Meilisearch-laravel-scout: Linter
We should have a linter so that everyone can easily contribute without messing the code 🙂
Steps:
- [ ] Add and configure the linter
- [ ] Add the command in README in the
Development Workflowpart - [ ] Add it in GitHub Action to be triggered on each PR
Laravel integration
curquiza
All 2 comments
You can quickly automate this step by using styleci, it is free for opensource
pet1330
on 24 Apr 2020
👍1
Thanks for the suggestion @pet1330! 😊
After discussing with the team, we would rather not to use styleCI since it needs to access to our organization content:
We are going to install a basic linter like php-cs-sniffer (already used in the PHP SDK).
curquiza
on 13 May 2020
👍2
Was this page helpful?
0 / 5 - 0 ratings
Related issues
curquiza
·
4Comments
marky291
·
6Comments
JexPY
·
8Comments
curquiza
·
17Comments
Kladislav
·
8Comments
Most helpful comment
Thanks for the suggestion @pet1330! 😊
After discussing with the team, we would rather not to use styleCI since it needs to access to our organization content:
We are going to install a basic linter like
php-cs-sniffer(already used in the PHP SDK).