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

Mkohm
Mkohm commented Oct 9, 2019

Would adding IDEs as a separate section be useful? For example IDEs like Jetbrains IDEA and Android Studio comes bundled with tools like idea-inspector and android lint. They also provide a lot of tools functionality for refactoring, extracting, reordering of methods, auto-formatting - using static analysis. Can create a PR for it if you think it is a good idea.

checkstyle
strkkk
strkkk commented Jul 10, 2019
$ cat checkstyle.xml
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN" "http://www.puppycrawl                           .com/dtds/configuration_1_2.dtd">
<module name="Checker">
    <module name="TreeWalker">
        <module name="OperatorWrapCheck">
        </module>
    </module>
</module>

$ cat Test.java
class A {
ndbroadbent
ndbroadbent commented Oct 14, 2019

Hello!

I think I've figured out a good way to get SimpleCov working with Spring. For my Rails application, preloading the app would require some classes in my ./app directory, and this would cause some inaccurate coverage to be reported (any lines that were required during preloading were missing in the coverage.) I figured out how to fix this issue by starting SimpleCov in config/spring.rb

ndclt
ndclt commented Jan 13, 2020

Steps to reproduce

  1. Save the following lines in a script
import pandas as pd

pd.DatetimeIndex([], name='time').tz_localize('UTC')
  1. run pylint on this script

Current behavior

Pylint returns:

[E1101(no-member), ] Instance of 'DatetimeIndex' has no 'tz_localize' member

Expected behavior

No error, tz_localize is a member of `DatetimeInde

Kristinita
Kristinita commented Mar 3, 2019

1. Summary

It would be nice, if HTMLHint documentation will contain arguments, why HTMLHint users must use specific rules.

2. Argumentation

“Must” in The script tag can not be used in head., Id and class value must meet some rules: underline, dash, hump. is not argument. The documentation does not show the reasons. Users should know “Why must”?

3. Example of expected behavior

mutant
dkubb
dkubb commented Sep 19, 2019

A feature request I have would be to add two flags to the mutant cli:

  • --skip-subject
  • --pending-subject

The --skip-subject flag should perform exactly the same as --ignore-subject, where it doesn't bother to mutate the subject nor run the subject's tests. This would be reserved for methods where attempting to mutate will cause an infinite loop and never finish.

The `--pending-

Splaktar
Splaktar commented Nov 11, 2018

AngularJS Material uses gulp to build everything, so we're looking at using https://github.com/ivogabe/gulp-typescript for our builds. It seems like something could be built similar to the webpack plugin in order to have the types added during the Gulp pipeline.

It's not clear if typewiz-node could help with this already or not. We don't have a single main.ts to point to. Should we do somethi

messerbill
messerbill commented Jan 16, 2019

I want to report a bug.

The bug is about "magic numbers" inside of strings written in backticks ( `)

SonarTS version: 1.8

Node.js version: 10.11.0

TypeScript version: 3.1.6

Reproducer

public static formatDate(dateString: string): string {
        let partials = dateString.split(".")
        partials = partials.map((item: string) => {
           
InvisibleSmiley
InvisibleSmiley commented Jan 22, 2020
  • PHP Depend version: 2.6.1
  • PHP Version: 7.3.12
  • Installation type: Composer
  • Operating System / Distribution & Version: Windows 10
  • no pdepend.xml / pdepend.xml.dist in directory to be checked with PHPMD

Current Behavior

Calling PHPMD (2.8.1) with a simple file takes ~6 seconds on Windows if your home directory is on a network share.

Expected Behavior

The performance shou

RogueScholar
RogueScholar commented Jan 10, 2020

First things first, thank you for creating and sharing a magnificent development tool, I'm severely chagrined to have waited to say that until I also wished to prevail on you for an expanded feature set; my mother raised me better than that.

Nevertheless, I was curious if you had ever entertained giving users the option to set an explicit threshold for aggregate file size reduction for a PR to

soullivaneuh
soullivaneuh commented Jan 24, 2018

It would be a great addition to be able to list useless dependencies. I mean an required dependency with no symbol usage at all.

Of course, it should be optional and the configuration file should accept packages to ignore.

Some packages could just provide configuration file or whatever not related to PHP directly, but used. This case would be a false positive.

Code quality apps

Coveralls

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

codelingo

Your Code, Your Rules - Automate code reviews with your own best practices

Scope for GitHub

Scope turns tests into the insights you need to ship higher quality software more frequently

Codecov

Group, merge and compare coverage reports

See more Code quality apps
You can’t perform that action at this time.