Conversation
b490478 to
f277c4d
Compare
|
Hi, thanks for this contribution, this is really nice clean up. We will check it, as we need to run some checks first. |
5292d10 to
f277c4d
Compare
|
Obviously this cleanup is not that simple... |
f277c4d to
177d4e0
Compare
|
Moved this code-duplication removal to its own PR, in the meantime, it should be possible to merge these remaining two simple cleanups. |
|
Could you describe why you are making this changes? Could you follow this guide https://github.com/decentraland/unity-renderer/blob/dev/.github/CONTRIBUTING.md |
|
I do believe I have been following the |
| # Set default behavior to automatically normalize line endings. | ||
| ############################################################################### | ||
| * text=auto | ||
| * text=auto eol=lf |
There was a problem hiding this comment.
On all modern OSes, modern editors (including VS Code) can deal with lf endings just fine, crlf is not needed.
If git is allowed to set the line endings to crlf before commits, this leads to discrepancies that are annoyingly useless at best.

And who knows what at worse ! Some of these files end up being used in a Docker environment, which will be expecting lf endings.
| test/**/*.js.map | ||
| dist | ||
| !packages/shared/world/runtime-7/sourcemap/source-map@0.7.4.js | ||
| /protocol-temp/ |
There was a problem hiding this comment.
This folder can remain if the build process is interrupted before the end.
Its brief appearance has also been confusing my own file monitoring process.
What does this PR change?