The Wayback Machine - https://web.archive.org/web/20220709155919/https://github.com/topics/code-quality
Skip to content
#

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 566 public repositories matching this topic...

reviewdog
TWiStErRob
TWiStErRob commented Apr 23, 2022

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:
![image](https://user-images.githubusercontent.com/2906988/164910306-4b4478c4-0

help wanted rules good first issue
PragTob
PragTob commented Jul 5, 2020

We have a problem: if files aren't loaded/required we don't have branch data. If they are added through track_files we give it 0/0 branches which we show as 100% coverage (all possible branches are covered).

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

Feature Branch Coverage good first issue
harupy
harupy commented Jul 4, 2022

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?

Good first issue Documentation 📗 Needs PR
aaronmorris
aaronmorris commented Mar 28, 2022

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
good first issue a:false-positive
orsinium
orsinium commented Jul 4, 2022

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

help wanted level:starter rule request good first issue

Code quality apps

Datree

YAML configs and K8s manifests validation tool

DeepScan

Advanced static analysis for automatically finding runtime errors in JavaScript code

TestQuality

The #1 Test Case Creation and Management for GitHub Workflows

Semgrep

Code scanning at ludicrous speed. Find bugs and enforce code standards

LGTM

Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review

DeepSource

Modern static analysis platform that helps engineering teams ship reliable and secure code

Coveralls

Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service

Codecov | Code Coverage

Automatic test report merging for all CI and languages into a single code coverage report directly into your pull request

abaplint

ABAP quality assurance and static analysis

Better Code Hub

A Benchmarked Definition of Done for Code Quality with BetterCodeHub

Codacy

Automated code reviews to help developers ship better software, faster

CommitCheck

CommitCheck ensures your commit messages are consistent and contain all required information

Sider

Automatically analyze pull request against custom per-project rulesets and best practices

See more Code quality apps