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 579 public repositories matching this topic...
-
Updated
Aug 9, 2022 - OCaml
-
Updated
Aug 9, 2022 - Rust
-
Updated
Jul 20, 2022 - JavaScript
-
Updated
Aug 9, 2022 - Java
-
Updated
Aug 9, 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 🐶 ...'
go
Expected 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
Aug 7, 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):
pass
pylint 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
Aug 8, 2022 - JavaScript
-
Updated
Jul 21, 2022 - TypeScript
-
Updated
Aug 9, 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
And, ideally, a configurable list of annotations denoting generated source or AST structures.
See #105 for the kind of issue this seeks to avoid, and the limitations of this fix.
-
Updated
Jul 30, 2022 - JavaScript
-
Updated
Aug 9, 2022 - Python
-
Updated
Oct 5, 2021 - TypeScript
-
Updated
Aug 2, 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
Aug 9, 2022
-
Updated
May 28, 2020 - Haskell
-
Updated
Aug 9, 2022 - TypeScript
-
Updated
Apr 14, 2022 - PHP
Code quality apps
Restyled.io
Restyle Pull Requests as they're opened
CommitCheck
CommitCheck ensures your commit messages are consistent and contain all required information
codebeat
Code review expert on demand. Automated for mobile and web
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
Code Climate
Automated code review for technical debt and test coverage
TestQuality
The #1 Test Case Creation and Management for GitHub Workflows
CodeScene
A quality visualization tool to identify and prioritize technical debt and evaluate your organizational efficiency
Datree
YAML configs and K8s manifests validation tool
LGTM
Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review
Codacy
Automated code reviews to help developers ship better software, faster
Code Review Doctor
Catch more Python and Django bugs during code review
abaplint
ABAP quality assurance and static analysis
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
Semgrep
Code scanning at ludicrous speed. Find bugs and enforce code standards
Sider
Automatically analyze pull request against custom per-project rulesets and best practices
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