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

[spiral/reactor] Fix Psalm issues and tests in Reactor #1002

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

msmakouz
Copy link
Member

Q A
Bugfix? ✔️
Breaks BC?
New feature?
  1. In the release of nette/php-generator v4.1.1, the omitEmptyNamespaces parameter was added with the default value true:
    https://github.com/nette/php-generator/releases/tag/v4.1.1
    Now if the namespace is empty or use is not used anywhere, the printer does not print it. The use of these elements has been added in tests.
  2. Psalm issues fixed.

@msmakouz msmakouz added this to the 3.9 milestone Oct 17, 2023
@msmakouz msmakouz self-assigned this Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (4376d23) 89.41% compared to head (0e95b00) 89.41%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1002   +/-   ##
=========================================
  Coverage     89.41%   89.41%           
  Complexity     6119     6119           
=========================================
  Files           810      810           
  Lines         17303    17303           
=========================================
  Hits          15471    15471           
  Misses         1832     1832           
Files Coverage Δ
src/Reactor/src/AbstractDeclaration.php 100.00% <ø> (ø)
src/Reactor/src/ClassDeclaration.php 100.00% <ø> (ø)
src/Reactor/src/EnumDeclaration.php 100.00% <ø> (ø)
src/Reactor/src/InterfaceDeclaration.php 100.00% <ø> (ø)
src/Reactor/src/Partial/Parameter.php 100.00% <ø> (ø)
src/Reactor/src/Partial/PromotedParameter.php 77.77% <ø> (ø)
src/Reactor/src/TraitDeclaration.php 54.54% <ø> (ø)
src/Reactor/src/Traits/TraitsAware.php 50.00% <100.00%> (ø)

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

@msmakouz msmakouz changed the title Fix Psalm issues and tests in Reactor [spiral/reactor] Fix Psalm issues and tests in Reactor Oct 17, 2023
@butschster butschster merged commit 0936229 into master Oct 17, 2023
9 checks passed
@msmakouz msmakouz deleted the bugfix/fix-reactor-tests branch October 17, 2023 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants