Yes, as pylama currently not requirement yet we will evaluate this later. Thanks!
Regards, Chasel > -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf Of > [email protected] > Sent: Wednesday, August 14, 2019 7:51 PM > To: [email protected]; Chiu, Chasel <[email protected]> > Cc: Feng, Bob C <[email protected]>; Gao, Liming <[email protected]>; > Leif Lindholm <[email protected]> > Subject: Re: [edk2-devel] [PATCH v2] BaseTools/Scripts: Add GetUtcDateTime > script. > > On 2019-08-14 04:21, Chiu, Chasel wrote: > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2067 > > > > A script that can return UTC date and time in ascii format which is > > convenient for patching build time information in any binary. > > > I know it's not a required tool to be run before committing, but could > you consider the following issues pylama reported, please? > > BaseTools/Scripts/GetUtcDateTime.py:1:1: E266 too many leading '#' for > block comment [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:10:1: E402 module level import not > at top of file [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:11:1: E402 module level import not > at top of file [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:12:1: E402 module level import not > at top of file [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:14:1: E302 expected 2 blank lines, > found 1 [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:29:14: E211 whitespace before '(' > [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:35:14: E211 whitespace before '(' > [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:38:14: E211 whitespace before '(' > [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:41:14: E211 whitespace before '(' > [pycodestyle] > BaseTools/Scripts/GetUtcDateTime.py:43:1: E305 expected 2 blank lines > after class or function definition, found 1 [pycodestyle] > > -- > Rebecca Cran > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45620): https://edk2.groups.io/g/devel/message/45620 Mute This Topic: https://groups.io/mt/32862249/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
