The Wayback Machine - https://web.archive.org/web/20220809233204/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 579 public repositories matching this topic...

jpsim
jpsim commented Sep 3, 2019

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

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

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

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

See more Code quality apps