Showing 243 of 379 total issues
Element (p.errornote) is overqualified, just use .errornote without element name. Open
Open
.errorlist, p.errornote + ul.errorlist,
- Exclude checks
Element (header#grp-header) is overqualified, just use #grp-header without element name. Open
Open
header#grp-header { position: static; }
- Exclude checks
Don't use IDs in selectors. Open
Open
#grp-content { top: 0; }
- Exclude checks
Don't use IDs in selectors. Open
Open
aside div#search { width: 200px; }
- Exclude checks
Adjoining classes: .tt-suggestion.tt-cursor Open
Open
.tt-suggestion.tt-cursor {
- Exclude checks
Expected COLON at line 72, col 26. Open
Open
color: #c00; inherit !important;
- Exclude checks
Don't use IDs in selectors. Open
Open
header#grp-header { position: static; }
- Exclude checks
Element (a.no-before::before) is overqualified, just use .no-before without element name. Open
Open
.panel-group .panel-title > a.no-before::before {
- Exclude checks
Using width with padding-left can sometimes make elements larger than you expect. Open
Open
padding-left: 1px;
- Exclude checks
Use of !important Open
Open
display: none !important;
- Exclude checks
Element (nav.navbar-default) is overqualified, just use .navbar-default without element name. Open
Open
nav.navbar-default { border-top-color: #3C8D2C; }
- Exclude checks
Rename function "makeExtension" to match the regular expression ^[a-z_][a-z0-9_]{2,}$. Open
Open
def makeExtension(**kwargs): # pylint: disable=invalid-name
- Read upRead up
- Exclude checks
Shared coding conventions allow teams to collaborate efficiently. This rule checks that all function names match a provided regular expression.
Noncompliant Code Example
With the default provided regular expression: ^[a-z_][a-z0-9_]{2,30}$
def MyFunction(a,b): ...
Compliant Solution
def my_function(a,b): ...
TODO found Open
Open
// TODO: remove the page reload above and add the new case to the list
- Exclude checks
TODO found Open
Open
// TODO: figure out how to reload above and add the new value to the page
- Exclude checks
TODO found Open
Open
// TODO: this entire file is mostly duplicate with testplans/js/mutable.js
- Exclude checks
Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed. Open
Open
def clean(self, value):
- Read upRead up
- Exclude checks
Cognitive Complexity is a measure of how hard the control flow of a function is to understand. Functions with high Cognitive Complexity will be difficult to maintain.
See
Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed. Open
Open
def update(
- Read upRead up
- Exclude checks
Cognitive Complexity is a measure of how hard the control flow of a function is to understand. Functions with high Cognitive Complexity will be difficult to maintain.
See
Merge this if statement with the enclosing one. Open
Open
if (not os.path.exists(filename)) or (not read_installation_id(filename)):
- Read upRead up
- Exclude checks
Merging collapsible if
statements increases the code's readability.
Noncompliant Code Example
if condition1: if condition2: # ...
Compliant Solution
if condition1 and condition2: # ...
BUG found Open
Open
keeping a reference to BUG-ID and trying to reconstruct the URL
- Exclude checks
TODO found Open
Open
// TODO: this can be moved to the back-end and provide the percentage there
- Exclude checks