The first submission to the Mozilla repository was rejected due to the usage of a banned/deprecated JavaScript Identifier: processNextEvent.
This was not on my code but on an external library (subprocess.jsm) .
There was already a warning on the automatic review about this issue, but since this is a module already used in other Mozilla Addons (e.g. Enigmail) I assumed it was OK to use it (apparently not).
This library is used before the test run to verify if the Robot Framework is installed on the system and issue a warning if it's not.
In the worst case scenario, I'll omit the verification and make this pre-requesite more evident in the documentation.
joao-carloto
commented
8 years ago
The first submission to the Mozilla repository was rejected due to the usage of a banned/deprecated JavaScript Identifier: processNextEvent. This was not on my code but on an external library (subprocess.jsm) . There was already a warning on the automatic review about this issue, but since this is a module already used in other Mozilla Addons (e.g. Enigmail) I assumed it was OK to use it (apparently not). This library is used before the test run to verify if the Robot Framework is installed on the system and issue a warning if it's not. In the worst case scenario, I'll omit the verification and make this pre-requesite more evident in the documentation.