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...
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.
$ 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 {
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
The documentation equates two kinds of attribute declarations:
explicit:
class Derp:
attribute: int = 1
@property
def property(self) -> int: ...and implicit:
class Derp:
def __init__(self):
self.attribute: int = 1Those two things are not about explicit v
| Q | A |
|---|---|
| Version | 0.17.1 |
| Bug? | maybe? |
| New feature? | no |
| Question? | no |
| Documentation? | probably? |
When run without a config option, the phpcsfixer2 task would fail with:
In ConfigurationResolver.php line 582:
[PhpCsFixer\ConfigurationException\InvalidConfigurationException (16)]
For multiple
Affects PMD Version:
All.
Rule:
ImmutableField
Description:
"Identifies private fields whose values never change once object initialization ends either in the declaration of the field or by a constructor. This helps in converting existing classes to becoming immutable ones."
Immutability, by defini
Steps to reproduce
- Save the following lines in a script
import pandas as pd
pd.DatetimeIndex([], name='time').tz_localize('UTC')- 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
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
https://docs.docker.com/docker-for-mac/osxfs-caching/
it seems that adding :cached to the /code, config, and cached volumes and :delegated to the /tmp/cc volume may improve performance on OSX machines at least.
Description
Currently the message body of reviewdog's annotations consists of static parts either reviewdog itself or the used tool has control of.
Example
For example, if i'm using pylint, i'm only able to adjust the line which comes from pylint's output via the --msg-template flag like this:
runner:
pylint:
cmd: pylint --msg-templatA 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-
-
Updated
Feb 3, 2020 - TypeScript
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
needs more docs
Thanks for the tool, which sounds interesting. This is just some drive-by new-user feedback in case it's helpful.
I read the docs, --help, and issues, then ran the project in two of my projects. I wasn't able to understand the output, or the theory of operation.
The shelltestrunner project has emacs style TAGS and vi style ctags files
-
Updated
Feb 3, 2020 - Shell
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) => {
Looking at the Dockerfile and the image's history (see below), it seems we can save quite a bit of space by switching to the multi-stage build pattern.
It should allow us to avoid having a 250MB+ intermediate layer in the image:
$ docker history jakzal/phpqa:alpine
IMAGE CREATED CREATED BY
Bug report
What's wrong
Simple usage of @contextlib.contextmanagerdecorator leads to WPS501. Example:
@contextlib.contextmanager
def _context(self, amount: int):
-
Updated
Feb 3, 2020 - TypeScript
- 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
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
-
Updated
Jan 13, 2020 - Vim script
We already use the implementation from typescript eslint for TS, however, the current implementation based on Java for JS behaves differently.
With the help of types (see MMF-1772), we can improve the rule for both for JS and
Code Smells
Landscape currently detects 43 code smells here
https://landscape.io/github/OrkoHunter/pep8speaks/17/messages/smell
Not all, but most of them should be easy to fix.
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.
-
Updated
Jan 23, 2020 - PHP
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
infer version v0.15.0
in my case, 0xFFFF or 0xFFFFFFFF is a common initializer and there are a lot of this semantics in our code. I would like to ignore those harmless. but if I turn off liveness checker, I worry about missing some truely harmful bugs. Does Infer provide some filtering option or method?