Skip to content

Add missing @since tags, add description of Locality provider#423

Merged
panilya merged 1 commit intodatafaker-net:mainfrom
panilya:add-since-tags
Oct 4, 2022
Merged

Add missing @since tags, add description of Locality provider#423
panilya merged 1 commit intodatafaker-net:mainfrom
panilya:add-since-tags

Conversation

@panilya
Copy link
Copy Markdown
Collaborator

@panilya panilya commented Oct 4, 2022

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 4, 2022

Codecov Report

Merging #423 (9b7ac15) into main (0916019) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #423      +/-   ##
============================================
+ Coverage     93.93%   93.95%   +0.02%     
- Complexity     2005     2006       +1     
============================================
  Files           222      222              
  Lines          4021     4021              
  Branches        384      384              
============================================
+ Hits           3777     3778       +1     
+ Misses          150      146       -4     
- Partials         94       97       +3     
Impacted Files Coverage Δ
src/main/java/net/datafaker/base/Locality.java 96.87% <ø> (ø)
src/main/java/net/datafaker/movie/FamilyGuy.java 100.00% <ø> (ø)
...java/net/datafaker/videogame/HeroesOfTheStorm.java 100.00% <ø> (ø)
.../main/java/net/datafaker/service/FakerContext.java 87.17% <0.00%> (+2.56%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bodiam
Copy link
Copy Markdown
Contributor

bodiam commented Oct 4, 2022

Oh, nice one, appreciate it!

@panilya panilya merged commit afeef5a into datafaker-net:main Oct 4, 2022
@panilya panilya deleted the add-since-tags branch October 4, 2022 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants