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 399 public repositories matching this topic...
-
Updated
Apr 29, 2020 - Rust
leftover from #4100 , detected at checkstyle/checkstyle#7775 (comment)
FileTabCharacter is still disabled at checkstyle_resources_checks.xml
When activated there bunch violations File contains tab characters (this is the first instance). [FileTabCharacter]
Some Inputs are created with tabs by design, so such files should be placed to suppress at `ch
-
Updated
May 3, 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 = 1
Those two things are not about explicit v
Q | A |
---|---|
Version | 1.8.0 |
Bug? | maybe |
New feature? | no |
Question? | no |
Documentation? | yes |
Related tickets | #152 |
This example from the documentation does not work for me in grumphp 1.8.0, it does not seem to resolve the variable $(pwd)
:
``
Affects PMD Version:
6.20.0
Rule:
PositionLiteralsFirstInComparisons
PositionLiteralsFirstInCaseInsensitiveComparisons
Description:
The above two rules must be
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
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.
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
-
Updated
May 14, 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 10, 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 29, 2018 - Shell
-
Updated
May 12, 2020 - Dockerfile
I am reading Code Smell Detection section of doc on npm:
When I click the link about no-dead-store
, it gives me a 404 page. Here is the page: https://github.com/SonarSource/SonarTS/blob/HEAD/sonarts-core/docs/rules/no-dead-store.md
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.
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.
-
Updated
Jun 25, 2019 - Vim script
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
May 12, 2020 - Ruby
Code quality apps
Datree
Policy enforcement solution for confident and compliant code
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
codebeat
Code review expert on demand. Automated for mobile and web
Imgbot
A GitHub app that optimizes your images
Code Climate
Automated code review for technical debt and test coverage
Restyled.io
Restyle Pull Requests as they're opened
Codacy
Automated code reviews to help developers ship better software, faster
TestQuality
Modern, powerful, test plan management
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
Codecov
Group, merge and compare coverage reports
Sider
Automatically analyze pull request against custom per-project rulesets and best practices
LGTM
Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
CodeFactor
Automated code review for GitHub
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?