Nox isn't linting tests #4697

Activity

1. theacodes

   added

   priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.Important issue which blocks shipping the next release. Will be fixed prior to next release.

   testing

   type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

   on Jan 4, 2018
2. theacodes

   self-assigned this

   on Jan 4, 2018
3. tseaver commented on Jan 8, 2018

   

   tseaver

   on Jan 8, 2018

   Contributor

   More actions

   @jonparrott Is this really either "P1" or a bug?
4. theacodes

   added

   priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.

   and removed

   priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.Important issue which blocks shipping the next release. Will be fixed prior to next release.

   on Jan 8, 2018
5. theacodes commented on Jan 8, 2018

   

   theacodes

   on Jan 8, 2018

   ContributorAuthor

   More actions

   I'm gonna drop it to a p2 but this is definitely a bug.
6. chemelnucfin commented on Jan 15, 2018

   

   chemelnucfin

   on Jan 15, 2018

   Contributor

   More actions

   also refer to #3418 which I will close
7. dhermes commented on Jan 15, 2018

   

   dhermes

   on Jan 15, 2018

   Contributor

   More actions

   @chemelnucfin Note that #3418 also says pylint should be turned back on (it is on, but doesn't cause test failures).
8. theacodes

   closed this as completedin #4921on Feb 24, 2018