arrow-py / arrow

🏹 Better dates & times for Python
https://arrow.readthedocs.io
Apache License 2.0
8.71k stars 673 forks source link

Add Maltese locale #972

Closed cyriaka90 closed 3 years ago

cyriaka90 commented 3 years ago

Pull Request Checklist

Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:

Description of Changes

Adds Maltese locale. :) Also fixes two white spaces in Malay locale.

codecov[bot] commented 3 years ago

Codecov Report

Merging #972 (5a46113) into master (06fe693) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #972   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         2036      2054   +18     
  Branches       328       330    +2     
=========================================
+ Hits          2036      2054   +18     
Impacted Files Coverage Δ
arrow/locales.py 100.00% <100.00%> (ø)

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 06fe693...5a46113. Read the comment docs.

anishnya commented 3 years ago

@systemcatch Assuming #970 gets merged in, this locale should be able to support dehumanize.

jadchaar commented 3 years ago

@anishnya does @cyriaka90 need to add the locale to any dehumanize fixtures?

anishnya commented 3 years ago

@jadchaar #970 hasn't been pulled in at this moment, so currently this locale wouldn't need to be added to the dehumanize fixtures.

jadchaar commented 3 years ago

Feel free to make any changes to other PRs @anishnya now that this is merged!