On Sat, Aug 18, 2018 at 8:43 PM William Roberts <bill.c.roberts@xxxxxxxxx> wrote: > > Im assuming with your attention on the python side of the house we're going to see a lot of > formatting change patches heading the mailing list. I don't have any problems with them. > > Are you using some formatter for these, if so which one is it? Is it flake8 still? I did not use a "formatter" here (I do not consider 2to3 to be a formatter). I only looked at pep8 and pylint warnings and modified the code with search-and-replace commands on the file (using regular expressions). I usually do not like large changes which reindent a file or add spaces, because it increases the amount of work needed to backport a later bugfix to released versions, for package maintainers. Nevertheless when the modified file is not used "in production" (only when debugging issues or when testing things), as is the case here, I prefer cleaning up the code. > We should probably document that patches should be sent formatted, ie if a patch introduces a delta > after applying the patch and running <formatter>, it's an issue. I upstreamed scripts/run-flake8 with this in mind ;) (and this is why I added it to Travis-CI tests too). And one of the reasons that led me to disable many warnings was to preserve some flexibility in order not to make submitting patches too hard for new contributors. In the next days, I plan to send some more patches to fix some formatting issues I consider important (for example removing semicolons at the end of Python statements), but I will not send patches which modify the spaces around operators in files where there could be bugs which would need to be backported. As always, thanks for your review! Nicolas PS: I am also playing with clang's static analyzer and I am currently testing a CircleCI configuration which runs it on every push (using scripts/run-scan-build) and publishes the results as HTML build artifacts. I will submit a proper patch for this once it is stable enough and once all the most important issues reported by the analyzer are fixed. _______________________________________________ Selinux mailing list Selinux@xxxxxxxxxxxxx To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx. To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.
