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.

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

Detect the original file newline style and apply it to rendered template #1159

Conversation

franciscomoma
Copy link
Contributor

Hello! I've seen that this issue was still open, and I decided to fix it detecting the original file ending style and applying it to the rendered new file.

I've also added a .gitattributes file to set CRLF to files that ends with *crlf.txt.

@dchimeno
Copy link

This solves my problem with Windows, any way this could be merge?

@tiangolo
Copy link

I think this is bitting several Windows users for some of the Cookiecutter project generators I have: tiangolo/full-stack-fastapi-couchbase#8, tiangolo/full-stack-fastapi-template#13

I guess this is probably bitting many more users of other Cookiecutters.

Is there a chance a maintainer can take a look at this PR? Is there any help we can provide?

@Hickstone
Copy link

We are also affected by this. This would solve many problems with generated unix files on windows.

@michaeljoseph michaeljoseph added the needs-review PR Only: This PR require review from other developer label Nov 4, 2019
@insspb insspb added the 1.8.0 label Dec 22, 2019
@insspb insspb added this to In progress in 1.8.0 Release via automation Dec 22, 2019
@ssbarnea ssbarnea added this to the 1.8.0 milestone Mar 30, 2020
@ssbarnea ssbarnea removed the 1.8.0 label Mar 30, 2020
@cookiecutter cookiecutter deleted a comment from codecov-io Apr 25, 2020
@insspb insspb modified the milestones: 1.8.0, 2.0.0 May 26, 2020
@insspb insspb added feature This issue/PR relates to major feature request. breaking-change Marks an important and likely breaking change. Require update for major version and removed enhancement This issue/PR relates to a feature request. labels May 27, 2020
@insspb insspb added WIP Work In Progress and removed needs-review PR Only: This PR require review from other developer labels May 27, 2020
@insspb insspb added this to In progress in 2.0.0 Release via automation May 27, 2020
@insspb insspb removed this from In progress in 1.8.0 Release May 27, 2020
@insspb insspb self-assigned this May 27, 2020
@insspb
Copy link
Member

insspb commented May 27, 2020

@franciscomoma
Thank you for contribution.

Merged to #1407
Will be in 2.0

@insspb insspb closed this May 27, 2020
2.0.0 Release automation moved this from In progress to Done May 27, 2020
@franciscomoma
Copy link
Contributor Author

@franciscomoma
Thank you for contribution.

Merged to #1407
Will be in 2.0

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Marks an important and likely breaking change. Require update for major version feature This issue/PR relates to major feature request. WIP Work In Progress
Projects
2.0.0 Release
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

8 participants