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

Documentation 1.8/2.0 update. #688

Merged
merged 2 commits into from
Feb 11, 2023
Merged

Documentation 1.8/2.0 update. #688

merged 2 commits into from
Feb 11, 2023

Conversation

bodiam
Copy link
Contributor

@bodiam bodiam commented Feb 11, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Feb 11, 2023

Codecov Report

Merging #688 (8395e0a) into main (f6f3c47) will increase coverage by 0.01%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main     #688      +/-   ##
============================================
+ Coverage     92.59%   92.61%   +0.01%     
- Complexity     2609     2610       +1     
============================================
  Files           283      283              
  Lines          5319     5319              
  Branches        574      574              
============================================
+ Hits           4925     4926       +1     
  Misses          248      248              
+ Partials        146      145       -1     
Impacted Files Coverage Δ
...in/java/net/datafaker/providers/base/Locality.java 64.81% <0.00%> (+1.85%) ⬆️

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

@bodiam bodiam merged commit 3d6315d into main Feb 11, 2023
@@ -7,7 +7,7 @@ First and foremost thanks to anyone who contributes, very much appreciated.
- Submit a PR with your change and if there are no comments, changes will be merged in
- If you're not sure about the change, raise an issue and have a discussion before spending time coding it up
- Try and make one logical change per PR. That is not make many changes in one PR. Submit multiple PRs instead
- Java 8 is our target version. If you need anything older than that, we recommend using [Java Faker](https://github.com/DiUS/java-faker) instead.
- Since Datafaker 2.x, Java 17 is our target version. If you need anything older than that, we recommend using the 1.x versions of Datafaker instead. Note that while the 1.x is stable and free of known bugs, this branch won't receive future updates.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I might be over thinking here so don't block on it.

But because 2.x is up coming shouldn't this be "For" or "Starting with" instead of "Since"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're probably right, I'm no native speaker. I'll update it, thanks!

* Remove usage Format Json from BaseFaker by @RVRhub in https://github.com/datafaker-net/datafaker/pull/663
* Include XML files in .gitattributes as text by @kingthorin in https://github.com/datafaker-net/datafaker/pull/664
* Add computer brands by @kingthorin in https://github.com/datafaker-net/datafaker/pull/665
* implement currently passing Xlint checks by @kingthorin in https://github.com/datafaker-net/datafaker/pull/667
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should drop this line for now 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, wellllll... I just hope it will come back soonish!

@kingthorin
Copy link
Collaborator

DOH, seems I was late.

@kingthorin kingthorin deleted the documentation_update branch February 11, 2023 12:11
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.

4 participants