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 578 public repositories matching this topic...
-
Updated
Jul 29, 2022 - OCaml
-
Updated
Jul 29, 2022 - Rust
-
Updated
Jul 20, 2022 - JavaScript
-
Updated
Jul 31, 2022 - Java
-
Updated
Jul 30, 2022 - OCaml
Many repositories need to fix, so please help if you like.
If you could help, it would be helpful if you could comment before starting the work not to overlapping.
Fix example
Run exit command after lint.
echo '::group:: Running golangci-lint with reviewdog 🐶 ...'
goExpected Behavior of the rule
UnnecessaryAbstractClass should have a concise description like other rules
Context
I just updated Detekt 1.19.0 to 1.20.0 and got this failure:
https://github.com/TWiStErRob/net.twisterrob.cinema/security/code-scanning/227
The message is just plain weird to be this long:
.
That math is "wrong" though here because there are branches but we don't know what they are. We should probably count total branches here as "unkown" and establish that in our "math unive
-
Updated
Jul 29, 2022 - JavaScript
Bug description
from abc import abstractmethod, ABCMeta
class WildAnimal(metaclass=ABCMeta):
@abstractmethod
def make_sound(self):
pass
class Panther(WildAnimal):
def make_sound(self):
raise NotImplementedError("Panther")
class PantherChild(Panther):
passpylint raise the following error against the code above?
Affects PMD Version: 6.42.0
Rule: ApexCRUDViolation
Description:
I'm getting the "Validate CRUD permission before SOQL/DML operation" error when updating a list of a custom object, but not on the individual.
Code Sample demonstrating the issue:
public static CustomObject__c updateCu-
Updated
Jul 31, 2022 - PHP
-
Updated
Jun 21, 2022
-
Updated
Jul 31, 2022 - JavaScript
-
Updated
Jul 21, 2022 - TypeScript
-
Updated
Jul 27, 2022 - Ruby
Forbid chaining `is`
Rule request
Thesis
Forbid ast.Is in ast.Compare.ops when it's size is not zero. For example:
a = None
a is None # True, as expected
a is None is True # False 🤔
a is None == True # False 🤔
a is None is None # True 🤯
Reasoning
From the AST perspective, is is an operator and can be chained. That can lead to unexpected results when the autho
Piranha transforms:
someConsumer.accept((x) -> {
if(exp.isTreated(STALE_FLAG)){
doSomething(x);
}
else{
domeSomethingElse(x);
}
-
Updated
Jul 30, 2022 - JavaScript
-
Updated
Jul 29, 2022 - Python
-
Updated
Oct 5, 2021 - TypeScript
-
Updated
Jul 15, 2022 - Dockerfile
I understand that optimisation of image/x-icon files has been added to Magick.NET (dlemstra/Magick.NET@c5e41c5). Are there any chances of this being added to Imgbot?
This has been requested before in #444, unfortunately, that issue has become stale without any comments whatsoever.
-
Updated
Jul 29, 2022
-
Updated
May 28, 2020 - Haskell
-
Updated
Jul 31, 2022 - TypeScript
-
Updated
Apr 14, 2022 - PHP
Code quality apps
Restyled.io
Restyle Pull Requests as they're opened
codebeat
Code review expert on demand. Automated for mobile and web
CommitCheck
CommitCheck ensures your commit messages are consistent and contain all required information
Sonatype Lift
Lift helps you find and fix your most elusive bugs so you can spend time writing great code, not debugging it
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
Codiga
Automate code reviews, analyze and scan code at each push/pull request
TestQuality
The #1 Test Case Creation and Management for GitHub Workflows
CodeFactor
Automated code review for GitHub
Imgbot
A GitHub app that optimizes your images
CodeScene
A quality visualization tool to identify and prioritize technical debt and evaluate your organizational efficiency
Datree
YAML configs and K8s manifests validation tool
Code Review Doctor
Catch more Python and Django bugs during code review
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
abaplint
ABAP quality assurance and static analysis

User @AnthonyMDev commented in #1674 about the lack of documentation around how SwiftLint merges nested configuration files. We should document the rules involved in config merging, either in our readme or a separate doc file.
This is a great possible "starter ticket" for users of SwiftLint that would like to contribute (😉 @AnthonyMDev). Looking at [Configuration+Merging.swift](https://github.c