Re: Extra checks for Pull Requests

Brian's Mail list account
 

Yes I get a nasty dong when it tries to auto update. Probably a broken link somewhere.
Brian

bglists@...
Sent via blueyonder.
Please address personal E-mail to:-
briang1@..., putting 'Brian Gaff'
in the display name field.
Newsgroup monitored: alt.comp.blind-users

----- Original Message -----
From: "zvonimir stanečić, 9a5dsz" <zvonimirek222@...>
To: <nvda-devel@groups.io>
Sent: Thursday, August 01, 2019 8:02 PM
Subject: Re: [nvda-devel] Extra checks for Pull Requests


Hi Reef,

To hijack the thread…

Why it’s not possible to download the latest alpha?

Somewhere we have 404 errors



From: nvda-devel@groups.io <nvda-devel@groups.io> On Behalf Of Reef Turner
Sent: Thursday, August 1, 2019 8:59 PM
To: nvda-devel@groups.io
Subject: [nvda-devel] Extra checks for Pull Requests



Hi everyone,



We have recently introduced new checks for Pull Requests on GitHub <https://github.com/nvaccess/nvda/pull/9958> .



Code contributors regularly have to deal with ill-defined and inconsistently enforced code style requirements. Code reviewers spend much of their time reporting minor issues, time that would be better spent looking for architectural problems / product issues / logic errors.



To improve this situation, in future PR builds the Flake8 linter will be run checking any new code that will be introduced with a PR. This can also be run with SCons using `scons lint base=origin/master`. Please see the `tests/lint/readme.md` file for more information.



As part of a PR build, the Flake8 linter will be run checking any new code that will be introduced with a PR. When Flake8 reports the does not comply with it’s configuration, a message from Appveyor will be added as a comment to the Pull Request, and the build will fail. Please note, that in this case the artifacts (PR build executable) will still be available if those steps were successful.



We have included an extension with the linter setup to allow tabs instead of spaces, and my find it necessary to disable other warnings, or introduce workarounds. However, in general we would like to stick as close to the default settings as possible.



Thanks for all your contributions!

--

Reef Turner
Software Developer



<https://www.nvaccess.org/> www.nvaccess.org

Facebook: https://www.facebook.com/NVAccess
Twitter: @NVAccess

<https://docs.google.com/uc?export=download&id=1ewvP2k8vO2yVx8f7qN46MLkUNt6pt8f_&revid=0B9qtAUOcqzA6dW5BQW55ZEp5UUptWjVYaXF5bnV4VC9qSGVRPQ>

Join nvda-devel@groups.io to automatically receive all group messages.