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 378 public repositories matching this topic...
-
Updated
Mar 30, 2020 - Rust
there was a lot of comments in PRs that test classes in suppressionxpathfilter should have 2 or more test methods.
This example clearly showed that maintainers tend to forgot this:
https://github.com/checkstyle/checkstyle/pull/7935/files#r399724307
We need Checkstyle to help. There is Check - RegexpSingleline.
Extended doc is at checkstyle/checkstyle#7714
We nee
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
Rule:
DoNotUseThreads
Description:
The documentation in here still shows Runnable as a problematic pattern. However, according to this issue this is no longer the case. Running the t
The pylint docs are currently hosted at pylint.pycqa.org. This doesn't support HTTPS as the certificate used is for *.readthedocs.io:
➜ curl -LIsv https://pylint.pycqa.org/
* Trying 104.208.221.96:443...
* TCP_NODELAY set
* Connected to pylint.pycqa.org (104.208.221.96) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certif
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
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
Apr 1, 2020 - TypeScript
-
Updated
Mar 29, 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
Rule request
Thesis
Sometimes you can find a code like this:
for ... in ...:
if ...:
return False # or True
return True # or False
And this ugly imperative logic can be easily replaced with pretty all()
or any()
calls.
Noti
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
-
Updated
Mar 3, 2020
-
Updated
May 29, 2018 - Shell
I want to report a bug.
TSLint version: 5.12.1
TSLint-SonarTS version: 1.9.0
Reproducer
// tslint:disable:no-identical-functions
// tslint:disable:no-duplicate-string
The no-commented-code
TSLint rule thinks this is commented code.
This seems t
-
Updated
Mar 31, 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
Hi,
Got an image where things go wrong:
https://github.com/nextcloud/nextcloud.com/pull/1248/files#diff-fd432c615a1a70b26b64e3e187a74a8e
https://github.com/nextcloud/nextcloud.com/pull/1248/files#diff-a4c83a7c5d4aa2a21624a18c168bfbab
Anything I can do to help/fix it?
if x, cachedExperiments.isTreated(forExperiment: ExperimentNamesSwift.test_experiment) {
// do something
}
is refactored into
if x, {
// do something
}
when isTreated
returns true
. The additional comma should be removed.
-
Updated
Jun 25, 2019 - Vim script
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
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.
-
Updated
Mar 28, 2020 - Ruby
Code quality apps
Restyled.io
Restyle Pull Requests as they're opened
Codacy
Automated code reviews to help developers ship better software, faster
Better Code Hub
A Benchmarked Definition of Done for Code Quality with BetterCodeHub
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
Code Climate
Automated code review for technical debt and test coverage
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
Sider
Automatically analyze pull request against custom per-project rulesets and best practices
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
Datree
Policy enforcement solution for confident and compliant code
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
DeepSource
Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python and Go
TestQuality
Modern, powerful, test plan management
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request
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?