Skip to content

Use https for schemaLocation - #650

Merged
snuyanzin merged 1 commit into
datafaker-net:mainfrom
snuyanzin:https
Jan 24, 2023
Merged

Use https for schemaLocation#650
snuyanzin merged 1 commit into
datafaker-net:mainfrom
snuyanzin:https

Conversation

@snuyanzin

Copy link
Copy Markdown
Collaborator

Use https like it is done for maven itself https://issues.apache.org/jira/browse/MNG-6778

@what-the-diff

what-the-diff Bot commented Jan 24, 2023

Copy link
Copy Markdown
  • The schema location was changed from http to https

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #650 (ce4a731) into main (15a2d88) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #650      +/-   ##
============================================
- Coverage     92.81%   92.77%   -0.05%     
- Complexity     2615     2623       +8     
============================================
  Files           280      281       +1     
  Lines          5371     5396      +25     
  Branches        587      589       +2     
============================================
+ Hits           4985     5006      +21     
- Misses          238      240       +2     
- Partials        148      150       +2     
Impacted Files Coverage Δ
.../main/java/net/datafaker/service/FakerContext.java 83.58% <0.00%> (-0.80%) ⬇️
...net/datafaker/internal/helper/SingletonLocale.java 75.00% <0.00%> (ø)
.../java/net/datafaker/service/FakeValuesService.java 84.95% <0.00%> (+0.02%) ⬆️
...c/main/java/net/datafaker/providers/base/Text.java 78.81% <0.00%> (+0.84%) ⬆️
...rc/main/java/net/datafaker/service/FakeValues.java 84.16% <0.00%> (+0.97%) ⬆️

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

@snuyanzin
snuyanzin merged commit a98c63b into datafaker-net:main Jan 24, 2023
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.

2 participants