Colin Seymour
Colin Seymour
> For Zmodel, the extension is expected to occur only once per repo. Using the search offered in the template, the result is 387 which is more than 200. 🤔...
Of course this all makes sense when you consider Prisma models will be for many DB tables and people prefer to break things up for easier maintenance (and ownership in...
I don't review PRs for popularity requirements etc after the initial review until I'm close to making the next release. The next release will be made sometime next week.
NOTE: +1 comments will be flagged. They add no value and don't influence things at all, especially when the PR isn't ready for merging due to failing tests. > @lildude...
You can save time by running `rake test` locally 😉
Guzzle and other dependencies has been updated in https://github.com/lildude/phpSmug/pull/93 which should address all of these.
> https://github.com/search?q=path:*.vto&type=code shows 2.7K now. Usage isn't an issue now. Lack of action on the initial review overall year ago is what is holding this PR back. This PR has...
Re-adding the grammar using `script/add-grammar` by replacing it with itself should resolve these test failures.
> @lildude thanks! I think it's fixed now (the tests pass in my codespace) 🤔 the `vendor/README.md` file update is missing (the tests should fail in a mo). Looks like...
> @lildude The `add-grammar` command removed this entry in the README.md file. I added again and the tests failed, so I assumed it should be removed ([here](https://github.com/github-linguist/linguist/pull/6733/commits/b13cd8e1db8b32901a7d15b1388b30aa8d5c92ad#diff-127a723f9b1317696bc8bcbc91190d0133ec55700142b019393f56c0cb2744e2)). > > Do...