Code review
Code review is systematic examination (sometimes referred to as peer review) of computer source code. It is intended to find mistakes overlooked in software development, improving the overall quality of software. Reviews are done in various forms such as pair programming, informal walkthroughs, and formal inspections.
Here are 229 public repositories matching this topic...
-
Updated
May 13, 2020
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
With a .stylelintrc.json
file with:
{
"extends": [
"stylelint-config-sass-guidelines",
"stylelint-config-prettier"
],
...
}
I get this error:
Error: Could not find "stylelint-config-sass-guidelines". Do you need a `configBasedir`?
at module.exports (/home/linters/app/node_modules/stylelint/lib/utils/configurationError.js:8:28)
at getModulePat
From @chrisdias:
If I start the auth dance, then press Cancel when asked if I want to let the extension open a URL, the status bar stays at Signing into github.com.... Since I cancelled, I expect it to revert back to Sign in to github.com.
Also (just noticed this) we say into and in to. Not sure which version is right or wrong, or if its based on context :).
-
Updated
May 19, 2020 - Python
Especially in the light of #151 , it would be great if people could just re-generate the whole Id file.
-
Updated
May 21, 2020 - Python
-
Updated
Aug 6, 2019
-
Updated
Feb 7, 2020 - JavaScript
-
Updated
Mar 9, 2020
IntelliJ version: 2018.3.2
Gerrit version: 1.1.1-146
If one adds a topic label in the push dialog and the laben contains whitespaces (i.e. "change ABC") the autofill of the push target is not working anylonger.
Steps to reproduce:
- open push dialog
- type two words in the "Topic" text field (i.e. "change ABC")
- click on checkbox "push to gerrit"
- click on checkbox "push to gerrit" t
-
Updated
Nov 28, 2017 - PHP
It might be useful to highlight tab characters (if only for Makefiles).
Also git allows to highlight trailing whitespaces, which is lost in diffr (because we trim all color codes).
-
Updated
Oct 8, 2017 - JavaScript
-
Updated
Nov 4, 2019 - Ruby
-
Updated
Mar 24, 2020 - CMake
What happened
A common workflow:
It would be nice if diff-goto-source
(C-c C-c
) worked when this is invoked from github-review-forge-pr-at-point
. All it needs to do is set the current directory to the git working tree in the diff-mode
buffer. In many cases it does not matter too much whether the user has the "correct" branch is checked out - it just needs to be close.
Caused by src-d/lookout#447 (comment)
It is needed to decide how to create PNG versions of SVG to be used in docs (because GitBook does not support SVG).
Currently it is done with a screenshot from mermaid view, but it does not scale and can not be integrated in the CI.
-
Updated
Mar 23, 2017 - Java
Currently it's assumed everything is in English.
This can lead to false positives (especially in French and Spanish).
It should be checked which language the document is written in
and typos should be fixed accordingly.
-
Updated
May 14, 2020 - Python
Split from #31
rails_best_practices
has format option, so we can set a checkstyle formatter.
I find a checkstyle formatter for rails_best_practices.
https://github.com/maroekun/rails_best_practices-formatter
I'm not sure this works fine with saddler.
Example for Brakeman
-
Updated
Sep 22, 2017
-
Updated
Dec 20, 2018 - JavaScript
- Wikipedia
- Wikipedia
Related Topics
gerritCode review apps
Mergify
Pull requests automation service
Code Climate
Automated code review for technical debt and test coverage
Review Notebook App
Code review for Jupyter Notebooks with visual diff & commenting
Pull Assistant
Pull Assistant analyses pull requests and provides the optimal way to perform code reviews
codebeat
Code review expert on demand. Automated for mobile and web
DeepScan
Advanced static analysis for automatically finding runtime errors in JavaScript code
AccessLint
Find accessibility issues in your pull requests
Better Code Hub
A Benchmarked Definition of Done for Code Quality with BetterCodeHub
Hound
Automated code reviews
WIP
DO NOT MERGE – as a service
DeepSource
Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python and Go
Datree
Policy enforcement solution for confident and compliant code
gitpod.io
Online IDE for GitHub
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request
I cannot figure out how to use properly danger on circleci 2.0 with workflows
Support I have a linear workflow where I bundle my app / install libraries (
build
job) and then run the tests in parallel. (I'm taking Ruby/Rspec as an example).How can I execute Danger to report test failures on all containers ?
Currently if a job fails on circleCI, the dependent jobs are not ran (ie if I