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 381 public repositories matching this topic...
-
Updated
Mar 30, 2020 - Rust
leftover from #4100 , detected at checkstyle/checkstyle#7775 (comment)
FileTabCharacter is still disabled at checkstyle_resources_checks.xml
When activated there bunch violations File contains tab characters (this is the first instance). [FileTabCharacter]
Some Inputs are created with tabs by design, so such files should be placed to suppress at `ch
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 | GrumPHP 0.17.2 |
| Bug? | yes |
| New feature? | no |
| Question? | no |
| Documentation? | no |
| Related tickets |
When using git_hook_variables parameter, you may also need to use variables inside your configured values. This seems to be impossible to do as the replaced $(EXEC_GRUMPHP_COMMAND) variable i
Rule:
DoNotUseThreads
Description:
The documentation in here still shows Runnable as a problematic pattern. However, according to this issue this is no longer the case. Running the t
The pylint docs are currently hosted at pylint.pycqa.org. This doesn't support HTTPS as the certificate used is for *.readthedocs.io:
➜ curl -LIsv https://pylint.pycqa.org/
* Trying 104.208.221.96:443...
* TCP_NODELAY set
* Connected to pylint.pycqa.org (104.208.221.96) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certif
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.
reviewdog/reviewdog#405 introduced the -tee option, but there is no mention of the feature in the readme. This would be helpful info for us who are trying to implement/debug this tool
In the following scenario, replacing the body of Child#one with super causes a SystemStackError when Child#one is being called. However mutant does not detect that this error kills the mutation that replaces the method's body with super. See https://github.com/matthias-g/mutant_super_bug for a running example.
class Parent
attr_reader :foo
def initialize(foo)
@foo = fo-
Updated
Apr 5, 2020 - TypeScript
-
Updated
Apr 4, 2020 - JavaScript
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
Rule request
Thesis
Sometimes you can find a code like this:
for ... in ...:
if ...:
return False # or True
return True # or FalseAnd this ugly imperative logic can be easily replaced with pretty all() or any() calls.
Noti
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
Mar 3, 2020
-
Updated
May 29, 2018 - Shell
I want to request a feature.
Typescript noUnusedParameters option ignores params beginning with underscore (understand that some people dislike this, but this is how it is). For such params that are being ignored it would seem preferable to use the optional parameter syntax over providing an unused default value. A default value makes it LESS clear that the parameter is actually not used.
-
Updated
Apr 2, 2020 - Dockerfile
- 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
Add watermark.
If Imgbot can add a watermark for images automatically, it will be better. The watermark can be configured in the config file.
if x, cachedExperiments.isTreated(forExperiment: ExperimentNamesSwift.test_experiment) {
// do something
}
is refactored into
if x, {
// do something
}
when isTreated returns true. The additional comma should be removed.
-
Updated
Jun 25, 2019 - 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.
-
Updated
Apr 2, 2020 - Ruby
Code quality apps
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
TestQuality
Modern, powerful, test plan management
Code Climate
Automated code review for technical debt and test coverage
Imgbot
A GitHub app that optimizes your images
CodeScene
A quality visualization tool to identify and prioritize technical debt and evaluate your organizational efficiency
codebeat
Code review expert on demand. Automated for mobile and web
Codacy
Automated code reviews to help developers ship better software, faster
Code Inspector
Code Quality, Code Reviews and Technical Debt evaluation made easy
Sider
Automatically analyze pull request against custom per-project rulesets and best practices
DeepSource
Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python and Go
Codecov
Group, merge and compare coverage reports
Datree
Policy enforcement solution for confident and compliant code
Restyled.io
Restyle Pull Requests as they're opened
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request
LGTM
Find and prevent zero-days and other critical bugs, with customizable alerts and automated code review
When using
mvn clean compile, everything goes well.But when using
infer -- mvn clean compile, report error:Error loading property file '****/${env}.propertiesTarget Project URL: https://github.com/shuzheng/zheng