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 405 public repositories matching this topic...
-
Updated
Jun 14, 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 14, 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 2, 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 = fo-
Updated
May 31, 2020 - JavaScript
WPS421 has pprint in the blacklist, however does not catch a call to pprint.pprint().
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 12, 2020
-
Updated
May 28, 2020 - Haskell
Given 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
Jun 1, 2020 - Dockerfile
-
Updated
May 29, 2018 - 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.
We currently refer to individual language analyzers as 'products': SonarJS, but they are not and have never been products. They are benefits delivered by SonarQube, SonarLint, and SonarCloud. Let's eliminate uses of 'SonarJS' to remove confusion. Specifically in
embedded docs & README.md. Use instead:
title "Code Quality and Security for JavaScript'
in text: 'analysis of JavaScript'
the Mark
-
Updated
Jun 25, 2019 - Vim script
-
Updated
Jun 12, 2020 - Ruby
Code quality apps
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
Datree
Policy enforcement solution for confident and compliant code
CodeFactor
Automated code review for GitHub
Restyled.io
Restyle Pull Requests as they're opened
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
Code Climate
Automated code review for technical debt and test coverage
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
CodeScene
A quality visualization tool to identify and prioritize technical debt and evaluate your organizational efficiency
LGTM
Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
codebeat
Code review expert on demand. Automated for mobile and web
Better Code Hub
A Benchmarked Definition of Done for Code Quality with BetterCodeHub
Codacy
Automated code reviews to help developers ship better software, faster
TestQuality
Modern, powerful, test plan management




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?