Conversation
hangy
left a comment
There was a problem hiding this comment.
Thanks for the suggestion! Good to see you're not using „dreiviertel zwölf“ 😜
src/Humanizer.Tests.Shared/Localisation/de/TimeToClockNotationTests.cs
Outdated
Show resolved
Hide resolved
src/Humanizer/Localisation/TimeToClockNotation/DeTimeOnlyToClockNotationConverter.cs
Outdated
Show resolved
Hide resolved
src/Humanizer/Localisation/TimeToClockNotation/DeTimeOnlyToClockNotationConverter.cs
Outdated
Show resolved
Hide resolved
|
@hangy you can add a culture like "de-DE-BY" with "dreiviertel zwölf" if you like 😜 |
hangy
left a comment
There was a problem hiding this comment.
I think the German changes are fine, but I don't know about changes to the public API.
| } | ||
| namespace Humanizer.Localisation.TimeToClockNotation | ||
| { | ||
| public class static German |
There was a problem hiding this comment.
Someone that knows the API policies of the project better than me should decide whether this change is appropriate, considering the existing public API.
There was a problem hiding this comment.
does it need to be public? if so can you add some docs that explain the use cases
There was a problem hiding this comment.
I would suggest to make it internal for now.
|
@SpocWeb can you rebase |
|
@SimonCropp sure, will try and also add some explanation to the contended Method(s) over the weekend, thanks for considering this. |
|
Hi, I don't know why the CI Builds fail. I am not allowed to look at the errors and locally it builds for me in all Configurations. |
Here is a checklist you should tick through before submitting a pull request:
mainbranch (more info below)fixes #<the issue number>build.cmdorbuild.ps1and ensure there are no test failures