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 241 public repositories matching this topic...
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
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
-
Updated
Jun 25, 2020
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
-
Updated
Jun 30, 2020 - CSS
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
Jul 7, 2020 - Python
Especially in the light of #151 , it would be great if people could just re-generate the whole Id file.
-
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
May 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
- Wikipedia
- Wikipedia
Related Topics
gerritCode review apps
codebeat
Code review expert on demand. Automated for mobile and web
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
Codacy
Automated code reviews to help developers ship better software, faster
Coveralls
Ensure that new code is fully covered, and see coverage trends emerge. Works with any CI service
Pull Assistant
Pull Assistant analyses pull requests and provides the optimal way to perform code reviews
Code Climate
Automated code review for technical debt and test coverage
gitpod.io
Online IDE for GitHub
Codecov
Group, merge and compare coverage reports
AccessLint
Find accessibility issues in your pull requests
Hound
Automated code reviews
Better Code Hub
A Benchmarked Definition of Done for Code Quality with BetterCodeHub
DeepSource
Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python, Go and Ruby
Review Notebook App
Code review for Jupyter Notebooks with visual diff & commenting
Lucidchart Connector
Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request



Description
On GitHub Enterprise without subdomain isolation, URL for GitHub pages becomes
https://example.com/pages/${username}/${reponame}instead of
https://pages.example.com/${username}/${reponame}ref: https://help.github.com/enterprise/2.15/user/articles/user-organizati