The Wayback Machine - https://web.archive.org/web/20220708153510/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

TestQuality

The #1 Test Case Creation and Management for GitHub Workflows

Better Code Hub

A Benchmarked Definition of Done for Code Quality with BetterCodeHub

Coveralls

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

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

codebeat

Code review expert on demand. Automated for mobile and web

Sonatype Lift

Lift helps you find and fix your most elusive bugs so you can spend time writing great code, not debugging it

DeepSource

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

Code Climate

Automated code review for technical debt and test coverage

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

See more Code quality apps