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

[Optimization] Update stat analyse level #42

Merged
merged 78 commits into from
May 21, 2023
Merged

[Optimization] Update stat analyse level #42

merged 78 commits into from
May 21, 2023

Conversation

ArtARTs36
Copy link
Owner

@ArtARTs36 ArtARTs36 commented Mar 25, 2023

Added

  • Add logs for HTTP Client
  • Validate BitBucket PR data

Changed

  • Update PHPStan level to 9

@ArtARTs36 ArtARTs36 changed the title Up phpstan Update stat analyse level May 20, 2023
# Conflicts:
#	composer.json
#	src/Presentation/Console/Application/ApplicationFactory.php
@ArtARTs36 ArtARTs36 changed the title Update stat analyse level [Optimization] Update stat analyse level May 21, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 21, 2023

Codecov Report

Patch coverage: 38.07% and project coverage change: -0.64 ⚠️

Comparison is base (e0aceb0) 50.75% compared to head (9ef74fa) 50.12%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #42      +/-   ##
============================================
- Coverage     50.75%   50.12%   -0.64%     
- Complexity      988     1116     +128     
============================================
  Files           266      271       +5     
  Lines          2900     3234     +334     
============================================
+ Hits           1472     1621     +149     
- Misses         1428     1613     +185     
Impacted Files Coverage Δ
src/Application/Linter/LinterFactory.php 0.00% <ø> (ø)
...pplication/Linter/TaskHandlers/LintTaskHandler.php 0.00% <ø> (ø)
.../Application/Rule/TaskHandlers/DumpTaskHandler.php 0.00% <0.00%> (ø)
src/Domain/Configuration/Config.php 0.00% <ø> (ø)
src/Domain/Note/AbstractNote.php 44.44% <0.00%> (-55.56%) ⬇️
src/Domain/Rule/Rules.php 0.00% <0.00%> (ø)
...ructure/Ci/Exceptions/CiInvalidParamsException.php 0.00% <0.00%> (ø)
.../Infrastructure/Ci/System/Bitbucket/API/Client.php 0.00% <0.00%> (ø)
...astructure/Ci/System/Bitbucket/API/PullRequest.php 66.66% <ø> (ø)
...xceptions/GivenInvalidPullRequestDataException.php 0.00% <0.00%> (ø)
... and 58 more

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ArtARTs36 ArtARTs36 merged commit fa71ffa into master May 21, 2023
1 check passed
@ArtARTs36 ArtARTs36 deleted the up-phpstan branch May 21, 2023 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants