On 15/09/2020 13:01, Chris Johns wrote:

On 15 Sep 2020, at 5:51 pm, Sebastian Huber<sebastian.hu...@embedded-brains.de> 
 wrote:

On 15/09/2020 04:33,chr...@rtems.org  wrote:

-    def get_includes(self):
+    def getIncludes(self):
I am not sure which coding style the file uses in general, but pylint complains 
about camelCase function names.
I did not run pylint, I only ran yapf. I followed the convention in the file. 
Is this wrong?
Before I used pylint, I used CamalCase for Python code, but then learned that pylint is not happy about this.

This code does need pylint run on it but I do not have the time.

Yes, no problem. It is just something I noticed. The patch is fine as it is.

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to