Skip to content

Used ToUpperInvariant so it doesn't run afoul of different culture settings#26

Open
droyad wants to merge 14 commits intojcoutch:masterfrom
OctopusDeploy:turkish
Open

Used ToUpperInvariant so it doesn't run afoul of different culture settings#26
droyad wants to merge 14 commits intojcoutch:masterfrom
OctopusDeploy:turkish

Conversation

@droyad
Copy link

@droyad droyad commented Nov 9, 2021

In the tr-TR culture, the letter i is capitalized as İ. This causes a problem when using Fri as a day of week value.

Since this library only accepts English weekdays, it's safe just to use the invariant culture.

# Conflicts:
#	NCrontab.Advanced.Tests/CronInstanceTests.cs
#	NCrontab.Advanced/Filters/StepFilter.cs
# Conflicts:
#	NCrontab.Advanced.Tests/CronInstanceTests.cs
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.

3 participants