Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

[refactor] complies with psr-2 #2006

Merged
merged 1 commit into from Oct 23, 2020

Conversation

rymiyamoto
Copy link
Contributor

I refactored some methods which were not psr-2 compliant.

Please review.

@codecov-commenter
Copy link

Codecov Report

Merging #2006 into master will decrease coverage by 0.20%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2006      +/-   ##
============================================
- Coverage     56.56%   56.36%   -0.21%     
  Complexity     2068     2068              
============================================
  Files           306      306              
  Lines          4849     4849              
============================================
- Hits           2743     2733      -10     
- Misses         2106     2116      +10     
Impacted Files Coverage Δ Complexity Δ
src/Faker/Provider/de_DE/Person.php 0.00% <0.00%> (-100.00%) 1.00% <0.00%> (ø%)
src/Faker/Provider/ne_NP/Person.php 0.00% <0.00%> (-50.00%) 2.00% <0.00%> (ø%)
src/Faker/Provider/uk_UA/Company.php 28.57% <0.00%> (-28.58%) 3.00% <0.00%> (ø%)
src/Faker/Provider/es_ES/Person.php 75.00% <0.00%> (-25.00%) 3.00% <0.00%> (ø%)
src/Faker/Provider/en_ZA/Address.php 50.00% <0.00%> (-25.00%) 4.00% <0.00%> (ø%)
src/Faker/Provider/it_IT/Address.php 75.00% <0.00%> (-25.00%) 4.00% <0.00%> (ø%)
src/Faker/Provider/me_ME/Address.php 44.44% <0.00%> (-22.23%) 4.00% <0.00%> (ø%)
src/Faker/Provider/en_UG/Person.php 60.00% <0.00%> (-20.00%) 5.00% <0.00%> (ø%)
src/Faker/Provider/hu_HU/Person.php 60.00% <0.00%> (-20.00%) 5.00% <0.00%> (ø%)
src/Faker/Provider/ko_KR/Address.php 80.00% <0.00%> (-20.00%) 5.00% <0.00%> (ø%)
... and 34 more

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 5337ce5...0da90d7. Read the comment docs.

@pimjansen pimjansen added this to the 1.9.2 milestone Oct 23, 2020
@pimjansen pimjansen merged commit 3ed1ec0 into fzaninotto:master Oct 23, 2020
@pimjansen
Copy link
Contributor

Thanks

GrahamCampbell pushed a commit to FakerPHP/Faker that referenced this pull request Oct 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants