-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Convert linebreak from CRLF to LF #18511
Conversation
PR #18511: Size comparison from 05a20b1 to 5389677 Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
We got 4 'gradlew.bat', 2 with LF, 2 with CRLF
It should be consistent about what to use. |
1184826
to
500a2d6
Compare
PR #18511: Size comparison from d79a0e3 to e710173 Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
That is very surprising to me. I wonder if this was somehow changed by mistake - I seem to remember gradle bits being a script/autogen. In any case, PR makes things metter, so merging. Thank you for taking the time to update and go through these. |
Problem
My IDE complain that a few files using CRLF linebreak
Change overview
Convert CRLF to LF
File
examples/ota-requestor-app/p6/matter-psoc6-mcuboot-bootloader.hex
also use CRLF, but it is a firmware file, so I'm not sure if it should be converted. Skipped this file.third_party/android_deps/gradlew.bat
./examples/tv-casting-app/android/App/gradlew.bat
and./src/android/CHIPTool/gradlew.bat
are shown in changes because git internally use LF for line-breaking when the file is managed by.gitattributes
. Once the file is checked out, it will be converted to CRLF automatically.Testing
Let CI verify it.