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.

Sign up for GitHub

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

Jump to bottom

Actualización #3

Merged
merged 92 commits into from
Sep 12, 2017
Merged

Actualización #3

merged 92 commits into from
Sep 12, 2017

Conversation

ogsilvaa
Copy link
Owner

Here is a checklist you should tick through before submitting a pull request:

  • Implementation is clean
  • Code adheres to the existing coding standards; e.g. no curlies for one-line blocks, no redundant empty lines between methods or code blocks, spaces rather than tabs, etc.
  • No ReSharper warnings
  • There is proper unit test coverage
  • If the code is copied from StackOverflow (or a blog or OSS) full disclosure is included. That includes required license files and/or file headers explaining where the code came from with proper attribution
  • There are very few or no comments (because comments shouldn't be needed if you write clean code)
  • Xml documentation is added/updated for the addition/change
  • Your PR is (re)based on top of the latest commits from the dev branch (more info below)
  • Link to the issue(s) you're fixing from your PR description. Use fixes #<the issue number>
  • Readme is updated if you change an existing feature or add a new one
  • Run either build.cmd or build.ps1 and ensure there are no test failures

aloisdg and others added 30 commits June 27, 2016 13:22
@aloisdg
Add Esperanto support to NumberToWords. Fixes #558
1fad764
@fishchisel
Fixed incorrect locale in readme
16dd003 
For example on `ToOrdinalWords` the Arabic example was incorrectly listed as 'Brazilian Portuguese'
@hazzik
Merge pull request #576 from fishchisel/fishchisel-patch-1
2f77816 
Fixed incorrect locale in readme
@hangy
Pass through the content of the "Letter, other" Unicode category.
65e123f 
fixes #578
@cameronsjo
Fix misspelling
cd5bbac 
Fixed misspelling of "Uper" to "Upper".
@hazzik
Merge pull request #581 from cameron-sjo/dev
6a0e77c 
Fix misspelling
@danielkatz
added translation for DateHumanize_Now
c8e0fa0
update for latest gv config
f49dc92
@master994
Add resource validation for DefaultFormatter to prevent "Value cannot…
71223f2 
… be null." exception on the .NET format.
@master994
Fix missing format value for the argument exception.
26ccdc0
@master994
Validate resource key before using .FormatWith extension.
199901d
@master994
Use string interpolation beside string format.
62399a6
@master994
Add summary for DefaultFormatter.
c7bf360
temp disable gitlink
75863fc
@cssho
Fix CONTRIBUTING.md of broken links.
7aacccc
@hazzik
Merge pull request #589 from cssho/fix_contributionmd_link
77ea9f1 
Fix CONTRIBUTING.md of broken links.
Merge pull request #586 from danielkatz/dev
b15f5e7 
added translation for DateHumanize_Now
Merge pull request #582 from hangy/issue578
289f97f 
Pass through the content of the "Letter, other" Unicode category.
@hangy
Squashed commit of the following:
4d6a015 
commit f9423de
Author: hangy <hangy@hangy.de>
Date:   Thu Dec 1 20:48:01 2016 +0100

    Update TimeSpanHumanizeTests.cs

commit 2e5dfe5
Author: hangy <hangy@hangy.de>
Date:   Thu Dec 1 20:37:50 2016 +0100

    Apparently I'm stupid and 7 days actually get humanized as 1 week.

commit 152a119
Author: hangy <hangy@hangy.de>
Date:   Wed Nov 30 23:50:51 2016 +0100

    Workaround for unneeded suffix: Just use the same value.

    The reason for this workaround is that at the method that decides if the suffix should be applied, the information about the time unit is not available. So it would either need to do some string magic and explicitly exclude the "TimeSpanHumanize_MultipleDays" resource key from adding the postfix, or most of the `TimeSpan` formatting code would need to be rewriten for Croatian or modified for all languages to support Croatian. Neither seems reasonable for a quick change.

commit 002cc7b
Author: hangy <hangy@hangy.de>
Date:   Wed Nov 30 23:12:57 2016 +0100

    Added some more tests. From what I've read, all days should resolve to "dana" for the Croatian language. (No "_DualTrialQuadral" postfix.")

commit c190e9d
Author: hangy <hangy@hangy.de>
Date:   Tue Nov 22 00:29:51 2016 +0100

    Add initial test for issue 597.
fix cmd build
e0ec345
update packages
c953863
Merge pull request #588 from Chrissx/issue-551
07bcaca 
Add resource value validation for DefaultFormatter
Merge pull request #569 from aloisdg/features/NumToWordsEo
1f518aa 
Add Esperanto support to NumberToWords. Fixes #558
Revert "Add Esperanto support to NumberToWords. Fixes #558"
8aba2af
Merge pull request #600 from Humanizr/revert-569-features/NumToWordsEo
ea5c0b5 
Revert "Add Esperanto support to NumberToWords. Fixes #558"
Merge pull request #598 from hangy/issue597
3399341 
Workaround for Croatian TimeSpan localization bug
Add Authenticode signing to binaries
97fa153
install signclient package
7e0d462
@hangy
Register Portuguese TimeSpan localisation as "pt".
a1d03cc 
According to #601, the localisation is good for
Portuguese in general, not just for Brazilian Portuguese.
I copied the unit test, so that we can prove that there's no regression.

Additionally, I copied the `OrginalizeTests`, because those were
already registered as "pt", not "pt-BR".

Note that `NumberToWords` is still only valid for pt-BR unless confirmed
otherwise.
@kinetiq
Syntax correction in readme.md
2fa67a7 
Very minor correction to the syntax. Named parameters require a :, not a =. 

(I found this when I copy/pasted the code and it didn't work without changes)
MaStr11 and others added 29 commits May 3, 2017 22:09
@MaStr11
Test for uniqueness of day sequences added.
4c64630
Include LV in main package
f5ee8e8
remove old project file
7464caa
Build script updates
05a8123
Add sourcelink
afe7928
fix sign client syntax
a10e589
Add support for long to ToQuantity
c840e39
Update Number to words to take a long
6f6c4ae
Add long.ToQuantity(ShowQuantityAs.Words)
d8441f1
Add extra public method for int and long
56af56b
Update public_api_approved.txt
e1fce66
Add tests for long.ToWords() and update XML comment
06871bb
Removing out of range test, correcting names of large numbers
125cb28
update to long
e88b07e
fix merge
8300823
Add ToOrdinalWords extension with gender overload
63818c2
Use localisation for ToOrdinalWords to allow for localised implementa…
b5e790d 
…tion and add readme section
Update XML docs for DateToOrdinalWords
3d71fd9
VS...grr.
638a16d
pdb is embedded now
2255698
@MaStr11
Incorporated review comments by @khellang, @M-Zuber, @jeremymeng. Mar…
5a38789 
…ked reviewed language with "Native speaker". Marked 'sv' as reviewed even confirmation is pending.
@MaStr11
Marked confirmed translations with "Native speaker".
dafcc6c
Update xUnit and fix test overload issue
cbdd7fa
Merge pull request #632 from MaStr11/FuzzyMonthYearsLogicForTimeSpanH…
0ab0d95 
…umanizeExtensions

Fuzzy month years logic for time span humanize extensions
@cmatskas
Added greek language resources as per #pr632
4c17268
Merge pull request #634 from cmatskas/cm-greeklang
4d1e9fe 
Added support for Greek language in resources
Merge pull request #633 from Humanizr/pr632
3bbcab0 
#632 Continued: Fuzzy month years logic for time span humanize extensions
v2.2 release notes
b30550e
Update package
8067889
@ogsilvaa ogsilvaa merged commit f8ccb7b into ogsilvaa:master Sep 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.