Coder Social home page Coder Social logo

ruby-saddler's Introduction

Saddler

Gem Version Build Status

To effectively use your lint messages!

checkstyle2anywhere, you can integrate any lint, security checker and tools with anywhere (e.g. GitHub Pull Request Review Comment.)

Pull Request Review Comment

Pull Request Review Comment

Pull Request Comment

Pull Request Comment

Usage

git diff -z --name-only origin/master \
 | xargs -0 rubocop-select \
 | xargs rubocop \
     --require rubocop/formatter/checkstyle_formatter \
     --format RuboCop::Formatter::CheckstyleFormatter \
 | checkstyle_filter-git diff origin/master \
 | saddler report \
    --require saddler/reporter/github \
    --reporter Saddler::Reporter::Github::PullRequestReviewComment

It works!

You can run this from any CI Service (e.g. circle-ci, travis-ci, jenkins, etc).

Reporters

Requirement

Set GITHUB_ACCESS_TOKEN=__your_access_token__ to your environment variable.

TravisCI

Travis CI: Encryption keys

$ gem install travis
$ travis encrypt -r <owner_name>/<repos_name> "GITHUB_ACCESS_TOKEN=<github_token>"

CircleCI

Environment variables - CircleCI

Demo

You can send pull request to repos below. Try this!

Examples

Articles

ja

vi

VS.

Easy to configure, only allow GitHub oAuth. Very quick response, because Hound uses not entire code base but pull request hook's payload. But Hound focuses on RuboCop and JavaScript linters wrapped by Rubygems. JavaScript libraries wrapped by Rubygems have code smells.

You can host own Hound. If you like caring hosted rails application.

Pronto is good application, and pronto influences saddler. Pronto's command seems simple, but this is "tightly-coupled" command. Pronto requires pronto-SOME-WRAPPER, and you should maintain wrapper scripts. Almost all of linters have their own command line interface. Why don't you use that direct?

FAQ

Q: Is there the way to share in the command line?

A: I think that sharing the command line is not a simple solution.

We can call saddler report multiple times. If we want to run Saddler only once, we can create "merged checkstyle file" before calling saddler report.

merge-checkstyle (command-a ...) (command-b ...) \
| saddler report ...

This requires merge-checkstyle command, I'm not sure that this command exists. I search "checkstyle" in rubygems, but I don't find such gem.

Q: Does Saddler support using both the text and Github reporters simultaneously?

A: Use tee. See Saddler::Reporter::Text and Saddler::Reporter::Github::PullRequestReviewComment.

Installation

Add this line to your application's Gemfile:

gem 'saddler'

And then execute:

$ bundle

Or install it yourself as:

$ gem install saddler

Development

After checking out the repo, run bin/setup to install dependencies. Then, run bin/console for an interactive prompt that will allow you to experiment. Run bundle exec saddler to use the code located in this directory, ignoring other installed copies of this gem.

To install this gem onto your local machine, run bundle exec rake install. To release a new version, update the version number in version.rb, and then run bundle exec rake release to create a git tag for the version, push git commits and tags, and push the .gem file to rubygems.org.

Contributing

  1. Fork it ( https://github.com/packsaddle/ruby-saddler/fork )
  2. Create your feature branch (git checkout -b my-new-feature)
  3. Commit your changes (git commit -am 'Add some feature')
  4. Push to the branch (git push origin my-new-feature)
  5. Create a new Pull Request

ruby-saddler's People

Contributors

jasnow avatar sanemat avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar

ruby-saddler's Issues

Diff and error exists, but not found error in github

https://travis-ci.org/packsaddle/example-travis_ci-commit/jobs/81002675
packsaddle/example-travis_ci-commit#1

E, [2015-09-18T12:04:23.079527 #2026] ERROR -- Saddler: options
E, [2015-09-18T12:04:23.079644 #2026] ERROR -- Saddler: {"options"=>{}, "debug"=>false, "verbose"=>false, "require"=>"saddler/reporter/github", "reporter"=>"Saddler::Reporter::Github::PullRequestReviewComment"}
E, [2015-09-18T12:04:23.079717 #2026] ERROR -- Saddler: input data
E, [2015-09-18T12:04:23.079774 #2026] ERROR -- Saddler: <?xml version='1.0'?>
<checkstyle>
  <file name='lib/example/travis_ci.rb'>
    <error column='120' line='7' message='Line is too long. [164/120]' severity='info' source='com.puppycrawl.tools.checkstyle.Metrics/LineLength'/>
  </file>
</checkstyle>
E, [2015-09-18T12:04:23.079857 #2026] ERROR -- Saddler: reporter
E, [2015-09-18T12:04:23.079913 #2026] ERROR -- Saddler: #<Saddler::Reporter::Github::PullRequestReviewComment:0x00000001c78ed0 @output=#<IO:<STDOUT>>>
/home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/response/raise_error.rb:16:in `on_complete': POST https://api.github.com/repos/packsaddle/example-travis_ci-commit/pulls/1/comments: 404 - Not Found // See: https://developer.github.com/v3/pulls/comments/#create-a-comment (Octokit::NotFound)
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/response.rb:9:in `block in call'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/response.rb:57:in `on_complete'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/response.rb:8:in `call'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/middleware/follow_redirects.rb:73:in `perform_with_redirection'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/middleware/follow_redirects.rb:61:in `call'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/rack_builder.rb:139:in `build_response'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/connection.rb:377:in `run_request'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/faraday-0.9.1/lib/faraday/connection.rb:177:in `post'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/sawyer-0.6.0/lib/sawyer/agent.rb:94:in `call'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/connection.rb:154:in `request'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/connection.rb:28:in `post'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/octokit-4.1.0/lib/octokit/client/pull_requests.rb:211:in `create_pull_request_comment'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/client.rb:53:in `create_pull_request_review_comment'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/pull_request_review_comment.rb:29:in `block in report'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/pull_request_review_comment.rb:28:in `each'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/pull_request_review_comment.rb:28:in `report'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-0.1.1/lib/saddler/cli.rb:37:in `report'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/saddler-0.1.1/exe/saddler:5:in `<top (required)>'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/bin/saddler:23:in `load'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/bin/saddler:23:in `<main>'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/bin/ruby_executable_hooks:15:in `eval'
    from /home/travis/.rvm/gems/ruby-1.9.3-p551/bin/ruby_executable_hooks:15:in `<main>'

Command for generate script

$ saddler generate rubocop #=> generate bin/build-rubocop.sh
$ saddler generate eslint #=> generate bin/build-eslint.sh

API rate limit exceeded for 52.0.240.122.

jser/jser.github.io#158
https://travis-ci.org/jser/jser.github.io/builds/80978812

E, [2015-09-18T08:56:46.240274 #2389] ERROR -- Saddler: options
E, [2015-09-18T08:56:46.240480 #2389] ERROR -- Saddler: {"options"=>{}, "debug"=>false, "verbose"=>false, "require"=>"saddler/reporter/github", "reporter"=>"Saddler::Reporter::Github::PullRequestReviewComment"}
E, [2015-09-18T08:56:46.240613 #2389] ERROR -- Saddler: input data
E, [2015-09-18T08:56:46.240710 #2389] ERROR -- Saddler: <?xml version='1.0' encoding='UTF-8'?><checkstyle version='4.3'><file name='/home/travis/build/jser/jser.github.io/_i18n/ja/_posts/2015/2015-09-16-react-0.14-eslint-1.4-es5.md'/></checkstyle>
E, [2015-09-18T08:56:46.240834 #2389] ERROR -- Saddler: reporter
E, [2015-09-18T08:56:46.240927 #2389] ERROR -- Saddler: #<Saddler::Reporter::Github::PullRequestReviewComment:0x000000014395d0 @output=#<IO:<STDOUT>>>
/home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/response/raise_error.rb:16:in `on_complete': GET https://api.github.com/repos/jser/jser.github.io/pulls/158/comments: 403 - API rate limit exceeded for 52.0.240.122. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.) // See: https://developer.github.com/v3/#rate-limiting (Octokit::TooManyRequests)
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/response.rb:9:in `block in call'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/response.rb:57:in `on_complete'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/response.rb:8:in `call'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/middleware/follow_redirects.rb:73:in `perform_with_redirection'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/middleware/follow_redirects.rb:61:in `call'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/rack_builder.rb:139:in `build_response'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/connection.rb:377:in `run_request'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/faraday-0.9.1/lib/faraday/connection.rb:140:in `get'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/sawyer-0.6.0/lib/sawyer/agent.rb:94:in `call'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/connection.rb:154:in `request'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/connection.rb:84:in `paginate'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/octokit-4.1.0/lib/octokit/client/pull_requests.rb:172:in `pull_request_comments'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/client.rb:39:in `pull_request_review_comments'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/saddler-reporter-github-0.1.4/lib/saddler/reporter/github/pull_request_review_comment.rb:16:in `report'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/saddler-0.1.1/lib/saddler/cli.rb:37:in `report'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
    from /home/travis/.rvm/gems/ruby-2.2.0/gems/saddler-0.1.1/exe/saddler:5:in `<top (required)>'
    from /home/travis/.rvm/gems/ruby-2.2.0/bin/saddler:23:in `load'
    from /home/travis/.rvm/gems/ruby-2.2.0/bin/saddler:23:in `<main>'
    from /home/travis/.rvm/gems/ruby-2.2.0/bin/ruby_executable_hooks:15:in `eval'
    from /home/travis/.rvm/gems/ruby-2.2.0/bin/ruby_executable_hooks:15:in `<main>'
The command "sh test/travis-spellcheck.sh" exited with 1.

commit_id is not part of the pull request

sorry to be late
after #44 , I got error below.

E, [2016-05-15T11:26:17.345644 #12599] ERROR -- Saddler: reporter
E, [2016-05-15T11:26:17.345693 #12599] ERROR -- Saddler: #<Saddler::Reporter::Github::PullRequestReviewComment:0x007ffc73b3ab88 @output=#<IO:<STDOUT>>>
/Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/response/raise_error.rb:16:in `on_complete': POST https://api.github.com/repos/myuser/sampleapp/pulls/3537/comments: 422 - Validation Failed (Octokit::UnprocessableEntity)
Error summary:
  resource: PullRequestReviewComment
  code: custom
  field: commit_id
  message: commit_id is not part of the pull request // See: https://developer.github.com/v3/pulls/comments/#create-a-comment
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/response.rb:9:in `block in call'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/response.rb:57:in `on_complete'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/response.rb:8:in `call'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/middleware/follow_redirects.rb:73:in `perform_with_redirection'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/middleware/follow_redirects.rb:61:in `call'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/rack_builder.rb:139:in `build_response'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/connection.rb:377:in `run_request'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/faraday-0.9.2/lib/faraday/connection.rb:177:in `post'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/sawyer-0.7.0/lib/sawyer/agent.rb:94:in `call'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/connection.rb:154:in `request'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/connection.rb:28:in `post'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/octokit-4.3.0/lib/octokit/client/pull_requests.rb:211:in `create_pull_request_comment'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/client.rb:80:in `create_pull_request_review_comment'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/pull_request_review_comment.rb:34:in `block in report'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/pull_request_review_comment.rb:33:in `each'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/pull_request_review_comment.rb:33:in `report'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-1.0.0/lib/saddler/cli.rb:37:in `report'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-1.0.0/exe/saddler:5:in `<top (required)>'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/bin/saddler:23:in `load'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/bin/saddler:23:in `<main>'

and environmental variables of Git Pull Request Builder plugin are below

$ghprbActualCommit: b237f0688c1bfa04555a76e61d693af5e072f40f  
$ghprbActualCommitAuthor: Yasuhiro Shimizu  
$ghprbActualCommitAuthorEmail: [email protected]  
$ghprbPullDescription:  GitHub pull request '#3572' of commit b237f0688c1bfa04555a76e61d693af5e072f40f, no merge conflicts.  
$ghprbPullId: 3572  
$ghprbPullLink: https://github.com/myuser/sampleapp/pull/3572  
$ghprbPullTitle: Issue/3419 check wip  
$ghprbSourceBranch: issue/3419-check-wip  
$ghprbTargetBranch: master  
$sha1: origin/pr/3572/merge

result is the same if I send pr from forked repo, or original repo

Missing examples

I very much like your approach of decoupling the report runner from the underlying linters.

Would it be possible for you to share the command line to enable running Saddler on Rails Best Practices and Brakeman as well?

Also, does Saddler support using both the text and Github reporters simultaneously?

should exit 1 if rubocop error raised

I'm using Rubocop with Saddler like this in Rails project on CircleCI hooked by Github push.

When there are Rubocop error with my code, CircleCI build ends with no errors (while warning comments are posted on PR).
I think Saddler Reporter should exit 1 if there are something to post.

thoughts?

fatal: Not a valid object name (Git::GitExecuteError) on Jenkins

Hi, thanks for this great tool.
I got an error below on my Jenkins(with GitHub pull request builder plugin, and reporter is PullRequestReviewComment)

E, [2016-05-12T17:39:45.412533 #44380] ERROR -- Saddler: reporter
E, [2016-05-12T17:39:45.412593 #44380] ERROR -- Saddler: #<Saddler::Reporter::Github::PullRequestReviewComment:0x007f97fcb016c0 @output=#<IO:<STDOUT>>>
/Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/git-1.3.0/lib/git/lib.rb:937:in `command': git '--git-dir=/Users/myuser/.jenkins/jobs/liverpool-android-pull-request/workspace/.git' '--work-tree=/Users/myuser/.jenkins/jobs/liverpool-android-pull-request/workspace' cat-file '-t' ''  2>&1:fatal: Not a valid object name  (Git::GitExecuteError)
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/git-1.3.0/lib/git/lib.rb:166:in `object_type'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/git-1.3.0/lib/git/object.rb:301:in `new'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/git-1.3.0/lib/git/base/factory.rb:53:in `object'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-support-git-1.0.0/lib/saddler/reporter/support/git/repository.rb:72:in `tracking'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-support-git-1.0.0/lib/saddler/reporter/support/git/repository.rb:64:in `block in merging_object'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-support-git-1.0.0/lib/saddler/reporter/support/git/repository.rb:63:in `select'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-support-git-1.0.0/lib/saddler/reporter/support/git/repository.rb:63:in `merging_object'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-support-git-1.0.0/lib/saddler/reporter/support/git/repository.rb:52:in `merging_sha'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/client.rb:69:in `block in pull_request_patches'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/client.rb:68:in `each'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/client.rb:68:in `pull_request_patches'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-reporter-github-1.0.0/lib/saddler/reporter/github/pull_request_review_comment.rb:23:in `report'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-1.0.0/lib/saddler/cli.rb:37:in `report'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/saddler-1.0.0/exe/saddler:5:in `<top (required)>'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/bin/saddler:23:in `load'
    from /Users/myuser/.rbenv/versions/2.0.0-p247/bin/saddler:23:in `<main>'

after digging it for a while, it seems like target branch name was unseeable from saddler.
but when I run echo $ghprbTargetBranch, it prints master

any idea how to solve this?

vs. reviewdog

https://github.com/haya14busa/reviewdog

I think reviewdog is awesome. They has single binary, no more dependency hell and simple usage. Wow!
They uses original format and checkstyle! So, we can migrate to reviewdog very easily, I think. I don't migrate yet.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.