Skip to content
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

Fix code style issues, fix grumphp TypeError #32

Merged
merged 1 commit into from
Feb 5, 2022
Merged

Conversation

mougrim
Copy link
Owner

@mougrim mougrim commented Feb 5, 2022

Also add composer.lock, which was removed in #29.

@github-actions
Copy link

github-actions bot commented Feb 5, 2022

Unit Test Results

  3 files  ±0    3 suites  ±0   0s ⏱️ ±0s
  8 tests ±0    8 ✔️ ±0  0 💤 ±0  0 ±0 
24 runs  ±0  24 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 1a6d2bd. ± Comparison against base commit 2ef0806.

♻️ This comment has been updated with latest results.

@mougrim mougrim merged commit 29ebde6 into master Feb 5, 2022
@mougrim mougrim deleted the phpCsFixer branch February 5, 2022 14:49
@mougrim mougrim mentioned this pull request Feb 5, 2022
mougrim added a commit that referenced this pull request Feb 5, 2022
There are next changes:

- Minimum PHP version supported upgraded to 7.4 (#29)
- PHPUnit version upgraded to 9 (#29)
- Add run unit tests Github action (#30, #31)
- Remove travis (#30)
- Fix code style issues (#32)
- Fix by workaround grumphp issue with TypeError, see phpro/grumphp#957 (#32)
- Update friendsofphp/php-cs-fixer to 3 (#33)
- Add psalm (#34, #35)
- Add check cs github action (#36)
mougrim added a commit that referenced this pull request Feb 5, 2022
There are next changes:

- Minimum PHP version supported upgraded to 7.4 (#29)
- PHPUnit version upgraded to 9 (#29)
- Add run unit tests Github action (#30, #31)
- Remove travis (#30)
- Fix code style issues (#32)
- Fix by workaround grumphp issue with TypeError, see phpro/grumphp#957 (#32)
- Update friendsofphp/php-cs-fixer to 3 (#33)
- Add psalm (#34, #35)
- Add check cs github action (#36)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant