-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add full feature parity #71
Conversation
peterdeme
commented
Jan 19, 2022
•
edited
Loading
edited
- Add full feature parity
- Add super strict linter rules (including mandatory documentation and type hint ❗)
- added documentation where it was missing
- Fix linting issues
- removed ramsey/uuid package
- bumped packages
- small refactors
- ability to code in VS Code via Docker, plus added a small documentation for it in CONTRIBUTING.md
707ce22
to
0f615da
Compare
@@ -23,22 +27,23 @@ jobs: | |||
uses: shivammathur/setup-php@v2 | |||
with: | |||
php-version: ${{ matrix.php-versions }} | |||
extensions: mbstring, intl | |||
extensions: ast, mbstring, intl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ast
gives a performance boost to phan
|
||
- name: Quality | ||
if: matrix.php-versions == '7.4' | ||
run: vendor/bin/phan --force-polyfill-parser || true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
force-polyfill-parser
isn't needed anymore because of ast
is added above
'AvoidableGetterPlugin', | ||
'RemoveDebugStatementPlugin', | ||
'HasPHPDocPlugin', | ||
], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these plugins are amazing. i was able to catch and fix a few issues with it.
0f615da
to
1d5c0b8
Compare
1d5c0b8
to
444ffd7
Compare