Skip to content
This repository was archived by the owner on Jan 4, 2022. It is now read-only.

Fix: Use magic methods instead of magic properties#435

Merged
ergebnis-bot merged 1 commit intomainfrom
fix/method
Mar 30, 2021
Merged

Fix: Use magic methods instead of magic properties#435
ergebnis-bot merged 1 commit intomainfrom
fix/method

Conversation

@localheinz
Copy link
Member

@localheinz localheinz commented Mar 30, 2021

This pull request

  • uses magic methods instead of deprecated magic properties

Follows #434.

@localheinz localheinz self-assigned this Mar 30, 2021
@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #435 (21c6d2f) into main (4a77eb6) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 21c6d2f differs from pull request most recent head 4d6ee9c. Consider uploading reports for the commit 4d6ee9c to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##                main      #435   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        87        87           
===========================================
  Files             12        12           
  Lines            302       302           
===========================================
  Hits             302       302           
Impacted Files Coverage Δ Complexity Δ
src/DataProvider/StringProvider.php 100.00% <100.00%> (ø) 12.00 <0.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 4a77eb6...4d6ee9c. Read the comment docs.

@ergebnis-bot ergebnis-bot self-assigned this Mar 30, 2021
@ergebnis-bot ergebnis-bot merged commit 7c85925 into main Mar 30, 2021
@ergebnis-bot ergebnis-bot deleted the fix/method branch March 30, 2021 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants