Awesome
Danger SwiftLint ![Build Status](https://app.travis-ci.com/ashfurrow/danger-ruby-swiftlint.svg?branch=master)
A Danger Ruby plugin for SwiftLint that runs on macOS.
Installation
Add this line to your Gemfile:
gem 'danger-swiftlint'
SwiftLint also needs to be installed before you run Danger, which you can do via Homebrew or a Brewfile. On CI, this is done for you when the gem is installed.
Usage
The easiest way to use is just add this to your Dangerfile:
swiftlint.lint_files
By default danger-swiftlint will lint added and modified files.
swiftlint.config_file = '.swiftlint.yml'
swiftlint.binary_path = '/path/to/swiftlint'
swiftlint.max_num_violations = 20
swiftlint.lint_files
If you want the lint result shows in diff instead of comment, you can use inline_mode
option. Violations that out of the diff will show in danger's fail or warn section.
swiftlint.lint_files inline_mode: true
If you want different configurations on different directories, you can specify the directory. Note: Run swiftlint.lint_files
per specified directory then.
swiftlint.directory = "Directory A"
If you want lint errors to fail Danger, you can use fail_on_error
option.
swiftlint.lint_files fail_on_error: true
If you need to specify options for swiftlint
that can only be specified by command line arguments, use the additional_swiftlint_args
option.
swiftlint.lint_files additional_swiftlint_args: '--lenient'
By default, only files that were added or modified are linted.
It's not possible to use nested configurations in that case, because Danger SwiftLint lints each file on it's own, and by doing that the nested configuration is disabled. If you want to learn more details about this, read the whole issue here.
However, you can use the lint_all_files
option to lint all the files. In that case, Danger SwiftLint doesn't lint files individually, which makes nested configuration to work. It'd be the same as you were running swiftlint
on the root folder:
swiftlint.lint_all_files = true
swiftlint.lint_files
It's also possible to pass a block to filter out any violations after swiftlint has been run. Here's an example filtering out all violations that didn't occur in the current github PR, using the third party gem git_diff_parser
:
require 'git_diff_parser'
diff = GitDiffParser::Patches.parse(github.pr_diff)
dir = "#{Dir.pwd}/"
swiftlint.lint_files(inline_mode: true) { |violation|
diff_filename = violation['file'].gsub(dir, '')
file_patch = diff.find_patch_by_file(diff_filename)
file_patch != nil && file_patch.changed_lines.any? { |line| line.number == violation['line']}
}
Or, by passing the no_comment
parameter, you can completely manage the commenting of issues, warnings and errors yourself. An example usage might be using GitHub reviews, or custom filtering logic. Note: When this parameter is set to true
, all other parameters except files
and additional_swiftlint_args
are ignored.
swiftlint.lint_files(no_comment: true)
# Now, you can handle the combined warnings + errors, or each separately commenting
swiftlint.issues # contains combined warnings + errors
swiftlint.warnings # just warnings
swiftlint.errors # just errors
You can use the SWIFTLINT_VERSION
environment variable to override the default version installed via the rake install
task.
Or, if you are behind a proxy or using a package manager which does not allow you to go to github.com directly, you can manually download the SwiftLint package and skip the installation by setting the DANGER_SKIP_SWIFTLINT_INSTALL environment variable equal to YES. https://github.com/ashfurrow/danger-ruby-swiftlint/blob/7c33f610349f40fabd28746a6ee2576b0aed7307/ext/swiftlint/Rakefile#L8
Finally, if something's not working correctly, you can debug this plugin by using setting swiftlint.verbose = true
.
Contribution tips
Upgrading Swiftlint to the latest version
Follow this guide -> How-To-Upgrade-Swiftlint
Attribution
Original structure, sequence, and organization of repo taken from danger-prose by David Grandinetti.
License
MIT