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

Croatian lang support #1468

Merged
merged 10 commits into from
Jan 19, 2024
Merged

Croatian lang support #1468

merged 10 commits into from
Jan 19, 2024

Conversation

CerealKiller0807
Copy link
Contributor

@CerealKiller0807 CerealKiller0807 commented Jan 18, 2024

I have made things!

Checklist

Related issues

#1467

@lk-geimfari
Copy link
Owner

Don't forget to add attribute HR to mimesis.enums.Locale, please.

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

Merging #1468 (bf9b843) into dev (f899514) will increase coverage by 0.14%.
Report is 1 commits behind head on dev.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1468      +/-   ##
==========================================
+ Coverage   99.11%   99.26%   +0.14%     
==========================================
  Files          58       58              
  Lines        2041     2041              
==========================================
+ Hits         2023     2026       +3     
+ Misses         18       15       -3     

@lk-geimfari lk-geimfari changed the base branch from master to dev January 18, 2024 16:04
Copy link
Owner

Choose a reason for hiding this comment

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

This file is too big. I think it make sence to reduce the volume of data in this file.

Copy link
Owner

Choose a reason for hiding this comment

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

The size of this file should ideally not exceed 500-600 kilobytes; otherwise, the final size of the mimesis package will be too large, and it may become a problem to install it for people with a slow internet connection.

@lk-geimfari lk-geimfari merged commit 9c3ba68 into lk-geimfari:dev Jan 19, 2024
10 checks passed
lk-geimfari added a commit that referenced this pull request Jan 19, 2024
* Croatian lang support (#1468)

---------

Co-authored-by: David Gorup <[email protected]>
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