[edk2-devel] [PATCH v2 1/1] Add a .editorconfig file to tell editors basic formatting details

[Rebecca Cran]

On 9/8/23 10:21, Ard Biesheuvel wrote:

> On Fri, 8 Sept 2023 at 17:54, Michael D Kinney
> <michael.d.kinney at intel.com> wrote:
>> Acked-by: Michael D Kinney <michael.d.kinney at intel.com>
>>
> Thanks
>
> I can merge this but note that we'll need to convert it to CR-LF
> first. (My VIM editor with editorconfig integration seems perfectly
> happy with that)

Good point - I'll fix the line endings and merge it.

Thinking of line endings, I was wondering if we might want to make the following additions to BaseTools/Conf/gitattributes, along with setting "core.autocrlf=false" and "core.eol=crlf"?

* text=auto
*.png -text
*.pyd -text
*.sh                                 eol=lf
.gitmodules                          eol=lf
BaseTools/BinWrappers/PosixLike/*    eol=lf
BaseTools/BinPipWrappers/PosixLike/* eol=lf
BaseTools/BuildEnv                   eol=lf
BaseTools/Conf/diff.order            eol=lf

-- 
Rebecca Cran



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108454): https://edk2.groups.io/g/devel/message/108454
Mute This Topic: https://groups.io/mt/97321658/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-