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 418 public repositories matching this topic...
-
Updated
Jul 9, 2020 - Rust
discussed in checkstyle/checkstyle#7542 (comment)
we need to extend AbstractGoogleModuleTestSupport to allow load few modules from config file.
In referenced PR we need to load together for each test:
<module name="RightCurly">
<property name="id" value="RightCurlyAlone"/>
<property name="option" value="alone"/>
<property name="tokens"
-
Updated
Jun 26, 2020 - JavaScript
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
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
reviewdog/reviewdog#405 introduced the -tee option, but there is no mention of the feature in the readme. This would be helpful info for us who are trying to implement/debug this tool
https://docs.docker.com/docker-for-mac/osxfs-caching/
it seems that adding :cached to the /code, config, and cached volumes and :delegated to the /tmp/cc volume may improve performance on OSX machines at least.
-
Updated
Jun 30, 2020 - TypeScript
In the following scenario, replacing the body of Child#one with super causes a SystemStackError when Child#one is being called. However mutant does not detect that this error kills the mutation that replaces the method's body with super. See https://github.com/matthias-g/mutant_super_bug for a running example.
class Parent
attr_reader :foo
def initialize(foo)
@foo = foGiven input:
if(cond1 || (cond2 && experiments.isToggleEnabled(STALE_FLAG)) {
// do something
}
the output of PiranhaJava is
if(cond1 || (cond2)) {
// do something
}
The additional paranthesis around cond2 should be eliminated.
-
Updated
Jul 3, 2020 - JavaScript
WPS421 has pprint in the blacklist, however does not catch a call to pprint.pprint().
-
Updated
Jul 8, 2020
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
-
Updated
May 28, 2020 - Haskell
-
Updated
Jul 10, 2020 - Dockerfile
-
Updated
Jun 22, 2020 - Shell
I want to request a feature.
Typescript noUnusedParameters option ignores params beginning with underscore (understand that some people dislike this, but this is how it is). For such params that are being ignored it would seem preferable to use the optional parameter syntax over providing an unused default value. A default value makes it LESS clear that the parameter is actually not used.
Hi,
It will be great if it's allowed to customize the commit message of imgbot, so that we can use conventional commit format without failing commitlint check.
When we use a rule from eslint core, we don't control which node is reported, in the case of big nodes, the resulting highlighting is ugly in the UI. Knowing that 30 more rules (see SonarSource/SonarJS#1918) 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
-
Updated
Jun 25, 2019 - Vim script
-
Updated
Jul 8, 2020 - Ruby
Code quality apps
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
CodeFactor
Automated code review for GitHub
CodeScene
A quality visualization tool to identify and prioritize technical debt and evaluate your organizational efficiency
Imgbot
A GitHub app that optimizes your images
DeepSource
Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python, Go and Ruby
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
Datree
Policy enforcement solution for confident and compliant code
Better Code Hub
A Benchmarked Definition of Done for Code Quality with BetterCodeHub
Code Climate
Automated code review for technical debt and test coverage
LGTM
Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review
Sider
Automatically analyze pull request against custom per-project rulesets and best practices
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request
Restyled.io
Restyle Pull Requests as they're opened
codebeat
Code review expert on demand. Automated for mobile and web
Codecov
Group, merge and compare coverage reports


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?