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

Enhancement: Use Php74 rule set #542

Merged
merged 2 commits into from Jan 14, 2022
Merged

Enhancement: Use Php74 rule set #542

merged 2 commits into from Jan 14, 2022

Conversation

localheinz
Copy link
Member

@localheinz localheinz commented Jan 14, 2022

This pull request

  • uses the Php74 rule set
  • runs make coding-standards

@localheinz localheinz requested a review from ergebnis-bot as a code owner Jan 14, 2022
@localheinz localheinz self-assigned this Jan 14, 2022
@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #542 (1763e8c) into main (bb68471) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##                main      #542   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity         7         7           
===========================================
  Files              2         2           
  Lines             26        26           
===========================================
  Hits              26        26           
Impacted Files Coverage Δ
src/RuleSet/AbstractRuleSet.php 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb68471...1763e8c. Read the comment docs.

@localheinz localheinz merged commit 21f3eab into main Jan 14, 2022
15 checks passed
@localheinz localheinz deleted the feature/php74 branch Jan 14, 2022
localheinz added a commit that referenced this issue Jan 14, 2022
Enhancement: Use `Php74` rule set
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