turquoiseowl / i18n

Smart internationalization for ASP.NET
Other
556 stars 156 forks source link

#173 add new param on constructor web config setting service #180

Closed jonnybee closed 9 years ago

turquoiseowl commented 9 years ago

We've got issues again here with full-file modifications. Any idea what's going on?

jonnybee commented 9 years ago

My only guess is a change of line endings – unix vs windows style.

I noticed the same immediately after Clone of the repository.

I added the default .gitattributes in Visual Studio (in previous checkin) and this now specifies:

###############################################################################

Set default behavior to automatically normalize line endings.

###############################################################################

Normally – this implies that

(assuming GIT identifies the extension as text)

/Jonny

turquoiseowl commented 9 years ago

I'm thinking I might need to do something savage like what's outlined here:

https://help.github.com/articles/dealing-with-line-endings/#refreshing-a-repository-after-changing-line-endings

turquoiseowl commented 9 years ago

In the hope of sorting out this "Wall of Pink" issue, I've done the following:

  1. Set my local git setup so that core.autocrlf = true: git config --global core.autocrlf true
  2. Followed the instructions for refreshing the repo after this change as outlined here: https://help.github.com/articles/dealing-with-line-endings/#refreshing-a-repository-after-changing-line-endings

The steps at 2. did not cause any files to be changed, either in the repo or those checked out.

So I remain stumped as to why this PR has a wall of pink.

turquoiseowl commented 9 years ago

Jonny, I've tried the line-ending normalization again and it worked. I'd be grateful if you would re-re-submit this PR! Thanks.

turquoiseowl commented 9 years ago

Full details on what I did here:

https://github.com/turquoiseowl/i18n#line-endings

jonnybee commented 9 years ago

Will do. Have a look at this sample project too: https://github.com/jonnybee/fluent_i18n_demo

jonnybee commented 9 years ago

I make "pull" to my repo from v2.0 od a further "pull" to my working branch. The changes should already be reflected in the existing PR.

turquoiseowl commented 9 years ago

Excellent, thanks.