Intent to Ship: JS & Python linting at review phase

[Sylvestre Ledru]

Hello,


tldr: We’re adding JS & Python linting at review phase. See 
https://reviewboard.mozilla.org/r/214622/as example.


A few months ago, we announced automated C/C++ Static Analysis in MozReview 
.As 
the feedback has been very positive, we have been working on bringing the same 
kind of feature for JavaScript and Python.


Thanks to the hard work of Andrew Halberstadt, Mark Banner and others, this 
work has been much easier. Indeed, we are fully relying on outputs provided by 
Mozlint . 
JavaScript linting is based on eslint, Python on flake8.


The analysis is performed by our bot the following way:

  *

Listen to new review requests;

  *

Apply the patches to a local mozilla-central clone;

  *

Run the analysis;Here, mach lint will do all the hard work for us.

  *

Publish the results on Mozreview (or Phabricator)

These actions are usually performed in less than 10 minutes.


We hope this will make the life of reviewers easier and also avoid some 
unnecessary tree breakage because of linter jobs. Also, as long as new linters 
are part of the mach lintinfrastructure, these new verifications will be 
automatically enabled at review phase.


Note that, unlike C/C++ static analysis, Release Management does not manage the 
activation/deactivation of various checkers. This job is done by Andrew and 
Mark, so we will forward any mozlint-related issues to them.


Phabricator support will be enabled in the next few weeks as well.


Please report any issues here: http://bit.ly/2y9N9Vx.


In the future, we are planning to instrument Phabricator to evaluate the 
usefulness of our checkers.


Bastien, Jan & Sylvestre

___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

[desktop] Bugs logged by Desktop Release QA in the last 8 days

[Bogdan Maris]

Hello,

Here's the list of new issues found and filed by the Desktop Release QA team 
last week.
Additional details on the team's priorities last week, as well as the plans for 
the current week are available at: https://goo.gl/w3hQ4W

Bugs logged by Desktop Release QA in the last 8 days

* NEW - https://bugzil.la/1431674 - If a video is playing while you sign into a 
youtube account, the video freezes.
* NEW - https://bugzil.la/1431749 - NEW [Intermittent] Crash in 
nsACString_internal::Assign | nsACString_internal::Assign | 
mozilla::gmp::GetContentParentFromDone::Done

This is available as a Bugzilla bug list as well: https://mzl.la/2DqaKjV

Regards,
Bogdan (:bogdan_maris)
Desktop Release QA
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform