Code quality
Automate your code review with style, quality, security, and test‑coverage checks when you need them most. Code quality is intended to keep complexity down and runtime up.
Here are 370 public repositories matching this topic...
-
Updated
Feb 19, 2020 - Rust
$ cat checkstyle.xml
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN" "http://www.puppycrawl .com/dtds/configuration_1_2.dtd">
<module name="Checker">
<module name="TreeWalker">
<module name="OperatorWrapCheck">
</module>
</module>
</module>
$ cat Test.java
class A {
Hello!
I think I've figured out a good way to get SimpleCov working with Spring. For my Rails application, preloading the app would require some classes in my ./app directory, and this would cause some inaccurate coverage to be reported (any lines that were required during preloading were missing in the coverage.) I figured out how to fix this issue by starting SimpleCov in config/spring.rb
The documentation equates two kinds of attribute declarations:
explicit:
class Derp:
attribute: int = 1
@property
def property(self) -> int: ...and implicit:
class Derp:
def __init__(self):
self.attribute: int = 1Those two things are not about explicit v
| Q | A |
|---|---|
| Version | 0.17.1 |
| Bug? | maybe? |
| New feature? | no |
| Question? | no |
| Documentation? | probably? |
When run without a config option, the phpcsfixer2 task would fail with:
In ConfigurationResolver.php line 582:
[PhpCsFixer\ConfigurationException\InvalidConfigurationException (16)]
For multiple
Affects PMD Version:
All.
Rule:
ImmutableField
Description:
"Identifies private fields whose values never change once object initialization ends either in the declaration of the field or by a constructor. This helps in converting existing classes to becoming immutable ones."
Immutability, by defini
Steps to reproduce
- Save the following lines in a script
import pandas as pd
pd.DatetimeIndex([], name='time').tz_localize('UTC')- run pylint on this script
Current behavior
Pylint returns:
[E1101(no-member), ] Instance of 'DatetimeIndex' has no 'tz_localize' member
Expected behavior
No error, tz_localize is a member of `DatetimeInde
1. Summary
It would be nice, if HTMLHint documentation will contain arguments, why HTMLHint users must use specific rules.
2. Argumentation
“Must” in The script tag can not be used in head., Id and class value must meet some rules: underline, dash, hump. is not argument. The documentation does not show the reasons. Users should know “Why must”?
3. Example of expected behavior
-
Updated
Feb 18, 2020 - Ruby
-
Updated
Feb 19, 2020 - Go
A feature request I have would be to add two flags to the mutant cli:
--skip-subject--pending-subject
The --skip-subject flag should perform exactly the same as --ignore-subject, where it doesn't bother to mutate the subject nor run the subject's tests. This would be reserved for methods where attempting to mutate will cause an infinite loop and never finish.
The `--pending-
-
Updated
Feb 19, 2020 - TypeScript
-
Updated
Feb 17, 2020 - JavaScript
AngularJS Material uses gulp to build everything, so we're looking at using https://github.com/ivogabe/gulp-typescript for our builds. It seems like something could be built similar to the webpack plugin in order to have the types added during the Gulp pipeline.
It's not clear if typewiz-node could help with this already or not. We don't have a single main.ts to point to. Should we do somethi
needs more docs
Thanks for the tool, which sounds interesting. This is just some drive-by new-user feedback in case it's helpful.
I read the docs, --help, and issues, then ran the project in two of my projects. I wasn't able to understand the output, or the theory of operation.
The shelltestrunner project has emacs style TAGS and vi style ctags files
Bug report
What's wrong
Currently, this is an error in our linter:
359:8 WPS204 Found overused expression: []; used 8 times
print(*[], **{'@': 1}) # noqa: WPS517, WPS445
But, how can one remove this violation? Impossible.
We s
I want to report a bug.
SonarTS version: 1.9.0
Node.js version: v10.15.1
TypeScript version: 3.4.5
SonarQube version: N/A
SonarLint version and used IDE: No, vscode
TSLint version: 5.16.0
Rule key:
bool-param-default
Provide a default value for 'devMode' so that the logic of the function is more evident when this parameter
-
Updated
Feb 19, 2020 - Dockerfile
- PHP Depend version: 2.6.1
- PHP Version: 7.3.12
- Installation type: Composer
- Operating System / Distribution & Version: Windows 10
- no pdepend.xml / pdepend.xml.dist in directory to be checked with PHPMD
Current Behavior
Calling PHPMD (2.8.1) with a simple file takes ~6 seconds on Windows if your home directory is on a network share.
Expected Behavior
The performance shou
The imgbot branch needs to be deleted for a new set of images to be compressed.
This needs to happen before triggering a manual refresh so we should add this info by the button.
Bonus points for detecting there is already a branch open and showing that info or offering a delete action.
 will rely on eslint-core, this situation may happen again in the future.
Rules concerned
- S138 (already fixed in https://github.com/SonarSource/SonarJS/iss
Code Smells
Landscape currently detects 43 code smells here
https://landscape.io/github/OrkoHunter/pep8speaks/17/messages/smell
Not all, but most of them should be easy to fix.
Improve logging
-
Updated
Feb 5, 2020 - Ruby
-
Updated
Feb 16, 2020 - PHP
-
Updated
Feb 19, 2020 - PHP
Code quality apps
TestQuality
Modern, powerful, test plan management
CodeFactor
Automated code review for GitHub
codebeat
Code review expert on demand. Automated for mobile and web
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
infer version v0.15.0
in my case, 0xFFFF or 0xFFFFFFFF is a common initializer and there are a lot of this semantics in our code. I would like to ignore those harmless. but if I turn off liveness checker, I worry about missing some truely harmful bugs. Does Infer provide some filtering option or method?