Skip to content
#

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

octotree
djy0
djy0 commented Nov 19, 2018

Description

Open this page https://github.com/facebook/react/commit/5bce0ef10ab28c87e57dd96dbe1d8470f8d68569?diff=split and open octotree sidebar, you can see a horizontal scrollbar appear at the bottom

Environment (if bug)

  • Octotree version: 2.5.1

  • Browser & version: chrome 70

  • OS & version: windows 10

  • Screenshot, if any (drag an image here)

  • Console error log below

Startouf
Startouf commented Jun 21, 2018

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

fbartho
fbartho commented Sep 19, 2019

The GitHub Integration just fails with a GitHub Error Response if your Issue Comment gets too long.

It’d be great if we truncated the message, & added a [truncated] 📝?

Thoughts @orta ? I’m happy to implement this if you have advice!

emattias
emattias commented Jan 20, 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
RMacfarlane
RMacfarlane commented Oct 2, 2019

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 :).

mahoe
mahoe commented Jan 11, 2019

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
Wikipedia
Wikipedia

Related Topics

gerrit

Code review apps

Lucidchart Connector

Insert a public link to a Lucidchart diagram so team members can quickly understand an issue or pull request

Hound

Automated code reviews

AccessLint

Find accessibility issues in your pull requests

DeepScan

Advanced static analysis for automatically finding runtime errors in JavaScript code

Code Climate

Automated code review for technical debt and test coverage

Code Inspector

Code Quality, Code Reviews and Technical Debt evaluation made easy

Codacy

Automated code reviews to help developers ship better software, faster

DeepSource

Discover bug risks, anti-patterns and security vulnerabilities before they end up in production. For Python, Go and Ruby

Sider

Automatically analyze pull request against custom per-project rulesets and best practices

Pull Assistant

Pull Assistant analyses pull requests and provides the optimal way to perform code reviews

Codecov

Group, merge and compare coverage reports

WIP

DO NOT MERGE – as a service

Better Code Hub

A Benchmarked Definition of Done for Code Quality with BetterCodeHub

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