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

Add missing dependencies to dev #73

Merged
merged 4 commits into from
Apr 8, 2024

Conversation

godruoyi
Copy link
Owner

@godruoyi godruoyi commented Apr 8, 2024

No description provided.

The dependencies illuminate/contracts, predis/predis, ext-redis, and ext-swoole have been added to the "require-dev" section of composer.json file. This provides additional functionality for development purposes, including dependency injection, connection with Redis, and asynchronous tasks.
This commit modifies the fail-fast strategy in the static analysis workflow and removes the installation of 'predis/predis' and 'illuminate/contracts' packages across multiple workflow files. This results in a cleaner setup, less dependency on external packages, and a more effective fail-fast configuration. Now, the workflows will use a matrix strategy for the PHP version, targeting specifically PHP
@codecov-commenter
Copy link

codecov-commenter commented Apr 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (dec895d) to head (e8f849d).
Report is 2 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #73      +/-   ##
============================================
+ Coverage     98.71%   98.74%   +0.02%     
- Complexity       91       94       +3     
============================================
  Files             7        8       +1     
  Lines           234      239       +5     
============================================
+ Hits            231      236       +5     
  Misses            3        3              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@godruoyi godruoyi mentioned this pull request Apr 8, 2024
2 tasks
@godruoyi godruoyi merged commit ee1f3af into master Apr 8, 2024
5 checks passed
@godruoyi godruoyi deleted the lianbo/add-missing-dependencies-to-dev branch April 8, 2024 05:24
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.

None yet

2 participants