Saddler
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 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
$ 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!
- TravisCI
- CircleCI
Examples
- textlint
- RuboCop
- JSCS
- Use multiple reporters simultaneously
- Gradle android - Checkstyle, FindBugs, PMD, CPD, Android Lint
Articles
ja
- jser/jser.github.ioの記事をpull request時にLintする仕組み | Web Scratch
- 変更したファイルにrubocopやjscsを実行して pull requestに自動でコメントする – Saddler - checkstyle to anywhere
- CircleCI - Androidのコードを自動で解析し、GitHubのpull requestにコメントする - Qiita
- pull requestした差分にrubocopの結果を自動でコメントしてCircleCIをfailedにするよ - Qiita
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
- Fork it ( https://github.com/packsaddle/ruby-saddler/fork )
- Create your feature branch (
git checkout -b my-new-feature
) - Commit your changes (
git commit -am 'Add some feature'
) - Push to the branch (
git push origin my-new-feature
) - Create a new Pull Request