govuk-lint 0.1.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 601bc226ff1d94cefb7a28a277a41ac5bc480aa0
+  data.tar.gz: 8ceeef9992a92b8d4b888777e61d42c3c421a2c9
+SHA512:
+  metadata.gz: d72cf29292398a8ed146650dafea8ffe4e0af0ef46b5d865a85578c451b045a3e5d516e07751f6ee4a6cc139555c7cdef021a92eb359e6efdf533406b5794638
+  data.tar.gz: b1eaa941e439a5c8ba5b78cac85fcfc8f033a0b7ee0ce36326b2430f7320fcb6739985254bf07000b65f155e042e47d8d8479e9eb084cfacb88ac82f872ccde1

data/bin/govuk-lint-ruby

@@ -0,0 +1,10 @@
+#!/usr/bin/env ruby
+
+lib = File.expand_path("../../lib", __FILE__)
+$:.unshift lib unless $:.include?(lib)
+
+require "govuk/lint"
+
+cli = Govuk::Lint::CLI.new
+
+exit cli.run

data/configs/rubocop/all.yml

@@ -0,0 +1,6 @@
+inherit_from:
+  - gds-ruby-styleguide.yml
+  - other-lint.yml
+  - other-rails.yml
+  - other-style.yml
+  - other-metrics.yml

data/configs/rubocop/gds-ruby-styleguide.yml

@@ -0,0 +1,318 @@
+############
+## General
+############
+
+BlockAlignment:
+  Description: 'Align block ends correctly.'
+  Enabled: true
+
+CaseIndentation:
+  Description: Indentation of when in a case/when/[else/]end.
+  Enabled: true
+  IndentWhenRelativeTo: case
+  SupportedStyles:
+  - case
+  - end
+  IndentOneStep: false
+
+ElseLayout:
+  Description: 'Check for odd code arrangement in an else block.'
+  Enabled: true
+
+# Supports --auto-correct
+EmptyLineBetweenDefs:
+  Description: Use empty lines between defs.
+  Enabled: true
+  AllowAdjacentOneLineDefs: false
+
+EmptyLines:
+  Description: "Don't use several empty lines in a row."
+  Enabled: true
+
+EndAlignment:
+  Description: 'Align ends correctly.'
+  Enabled: true
+
+EndOfLine:
+  Description: 'Use Unix-style line endings.'
+  Enabled: true
+
+# Supports --auto-correct
+IndentationWidth:
+  Description: Use 2 spaces for indentation.
+  Enabled: true
+
+# Supports --auto-correct
+IndentationConsistency:
+  Description: Keep indentation straight.
+  Enabled: true
+
+LineLength:
+  Description: Limit lines to 80 characters.
+  Enabled: true
+  Max: 80
+
+# Supports --auto-correct
+SpaceAroundOperators:
+  Description: Use spaces around operators.
+  Enabled: true
+
+# Supports --auto-correct
+SpaceBeforeBlockBraces:
+  Description: Checks that the left block brace has or doesn't have space before it.
+  Enabled: true
+  EnforcedStyle: space
+  SupportedStyles:
+  - space
+  - no_space
+
+# Supports --auto-correct
+SpaceAfterSemicolon:
+  Description: Use spaces after semicolons.
+  Enabled: true
+
+# Supports --auto-correct
+SpaceAfterColon:
+  Description: Use spaces after colons.
+  Enabled: true
+
+# Supports --auto-correct
+SpaceAfterComma:
+  Description: Use spaces after commas.
+  Enabled: true
+
+# Supports --auto-correct
+SpaceInsideBrackets:
+  Description: No spaces after [ or before ].
+  Enabled: true
+
+# Supports --auto-correct
+SpaceInsideParens:
+  Description: No spaces after ( or before ).
+  Enabled: true
+
+Tab:
+  Description: No hard tabs.
+  Enabled: true
+
+# Supports --auto-correct
+TrailingWhitespace:
+  Description: Avoid trailing whitespace.
+  Enabled: true
+
+# Supports --auto-correct
+TrailingBlankLines:
+  Description: Checks trailing blank lines and final newline.
+  Enabled: true
+  EnforcedStyle: final_newline
+  SupportedStyles:
+  - final_newline
+  - final_blank_line
+
+## Syntax
+
+# Supports --auto-correct
+AndOr:
+  Description: Use &&/|| instead of and/or.
+  Enabled: true
+
+# Supports --auto-correct
+DefWithParentheses:
+  Description: Use def with parentheses when there are arguments.
+  Enabled: true
+
+For:
+  Description: Checks use of for or each in multiline loops.
+  Enabled: true
+  EnforcedStyle: each
+  SupportedStyles:
+  - for
+  - each
+
+IfUnlessModifier:
+  Description: Favor modifier if/unless usage when you have a single-line body.
+  Enabled: true
+  MaxLineLength: 80
+
+MethodCalledOnDoEndBlock:
+  Description: Avoid chaining a method call on a do...end block.
+  Enabled: true
+
+MethodCallParentheses:
+  Description: 'Do not use parentheses for method calls with no arguments.'
+  Enabled: true
+
+MultilineBlockChain:
+  Description: Avoid multi-line chains of blocks.
+  Enabled: true
+
+MultilineIfThen:
+  Description: Never use then for multi-line if/unless.
+  Enabled: true
+
+MultilineTernaryOperator:
+  Description: 'Avoid multi-line ?: (the ternary operator); use if/unless instead.'
+  Enabled: true
+
+NestedTernaryOperator:
+  Description: Use one expression per branch in a ternary operator.
+  Enabled: true
+
+OneLineConditional:
+  Description: Favor the ternary operator(?:) over if/then/else/end constructs.
+  Enabled: true
+
+# Supports --auto-correct
+ParenthesesAroundCondition:
+  Description: Don't use parentheses around the condition of an if/unless/while.
+  Enabled: true
+  AllowSafeAssignment: true
+
+# Supports --auto-correct
+RedundantReturn:
+  Description: Don't use return where it's not required.
+  Enabled: true
+  AllowMultipleReturnValues: false
+
+# Supports --auto-correct
+SpaceAfterMethodName:
+  Description: Never put a space between a method name and the opening parenthesis in
+    a method definition.
+  Enabled: true
+
+# Supports --auto-correct
+SpaceAroundEqualsInParameterDefault:
+  Description: Checks that the equals signs in parameter default assignments have or
+    don't have surrounding space depending on configuration.
+  Enabled: true
+  EnforcedStyle: space
+  SupportedStyles:
+  - space
+  - no_space
+
+UnlessElse:
+  Description: Never use unless with else. Rewrite these with the positive case first.
+  Enabled: true
+
+# Supports --auto-correct
+UnusedBlockArgument:
+  Description: Checks for unused block arguments.
+  Enabled: true
+
+## Naming
+
+ClassAndModuleCamelCase:
+  Description: Use CamelCase for classes and modules.
+  Enabled: true
+
+# Supports --auto-correct
+ClassMethods:
+  Description: Use self when defining module/class methods.
+  Enabled: true
+
+ClassVars:
+  Description: Avoid the use of class variables.
+  Enabled: true
+
+ConstantName:
+  Description: Constants should use SCREAMING_SNAKE_CASE.
+  Enabled: true
+
+MethodName:
+  Description: Use the configured style when naming methods.
+  Enabled: true
+  EnforcedStyle: snake_case
+  SupportedStyles:
+  - snake_case
+  - camelCase
+
+# Supports --auto-correct
+TrivialAccessors:
+  Description: Prefer attr_* methods to trivial readers/writers.
+  Enabled: true
+  ExactNameMatch: true
+  AllowPredicates: false
+  AllowDSLWriters: false
+  Whitelist:
+  - to_ary
+  - to_a
+  - to_c
+  - to_enum
+  - to_h
+  - to_hash
+  - to_i
+  - to_int
+  - to_io
+  - to_open
+  - to_path
+  - to_proc
+  - to_r
+  - to_regexp
+  - to_str
+  - to_s
+  - to_sym
+
+VariableName:
+  Description: Use the configured style when naming variables.
+  Enabled: true
+  EnforcedStyle: snake_case
+  SupportedStyles:
+  - snake_case
+  - camelCase
+
+## Exceptions
+
+# Supports --auto-correct
+RescueException:
+  Description: Avoid rescuing the Exception class.
+  Enabled: true
+
+## Collections
+
+# Supports --auto-correct
+AlignArray:
+  Description: Align the elements of an array literal if they span more than one line.
+  Enabled: true
+
+# Supports --auto-correct
+HashSyntax:
+  Description: 'Prefer Ruby 1.9 hash syntax { a: 1, b: 2 } over 1.8 syntax { :a => 1,
+    :b => 2 }.'
+  Enabled: true
+  Exclude:
+  - 'db/schema.rb'
+
+  EnforcedStyle: ruby19
+  SupportedStyles:
+  - ruby19
+  - hash_rockets
+
+TrailingComma:
+  Description: Checks for trailing comma in parameter lists and literals.
+  Enabled: false
+  EnforcedStyleForMultiline: comma
+  SupportedStyles:
+  - comma
+  - no_comma
+
+# Supports --auto-correct
+WordArray:
+  Description: Use %w or %W for arrays of words.
+  Enabled: true
+  MinSize: 0
+
+## Strings
+
+# Supports --auto-correct
+StringConversionInInterpolation:
+  Description: Checks for Object#to_s usage in string interpolation.
+  Enabled: true
+
+# Supports --auto-correct
+StringLiterals:
+  Description: Checks if uses of quotes match the configured preference.
+  Enabled: true
+  EnforcedStyle: double_quotes
+  SupportedStyles:
+  - single_quotes
+  - double_quotes

data/configs/rubocop/other-lint.yml

@@ -0,0 +1,131 @@
+AmbiguousOperator:
+  Description: >-
+     Checks for ambiguous operators in the first argument of a
+     method invocation without parentheses.
+  Enabled: false
+
+AmbiguousRegexpLiteral:
+  Description: >-
+    Checks for ambiguous regexp literals in the first argument of
+    a method invocation without parenthesis.
+  Enabled: false
+
+AssignmentInCondition:
+  Description: "Don't use assignment in conditions."
+  Enabled: false
+
+ConditionPosition:
+  Description: 'Checks for condition placed in a confusing position relative to the keyword.'
+  Enabled: false
+
+Debugger:
+  Description: 'Check for debugger calls.'
+  Enabled: false
+
+DeprecatedClassMethods:
+  Description: 'Check for deprecated class method calls.'
+  Enabled: false
+
+EmptyEnsure:
+  Description: 'Checks for empty ensure block.'
+  Enabled: false
+
+EmptyInterpolation:
+  Description: 'Checks for empty string interpolation.'
+  Enabled: false
+
+EndInMethod:
+  Description: 'END blocks should not be placed inside method definitions.'
+  Enabled: false
+
+EnsureReturn:
+  Description: 'Never use return in an ensure block.'
+  Enabled: false
+
+Eval:
+  Description: 'The use of eval represents a serious security risk.'
+  Enabled: false
+
+HandleExceptions:
+  Description: "Don't suppress exception."
+  Enabled: false
+
+InvalidCharacterLiteral:
+  Description: >-
+     Checks for invalid character literals with a non-escaped
+     whitespace character.
+  Enabled: false
+
+LiteralInCondition:
+  Description: 'Checks of literals used in conditions.'
+  Enabled: false
+
+LiteralInInterpolation:
+  Description: 'Checks for literals used in interpolation.'
+  Enabled: false
+
+Loop:
+  Description: >-
+     Use Kernel#loop with break rather than begin/end/until or
+     begin/end/while for post-loop tests.
+  Enabled: false
+
+ParenthesesAsGroupedExpression:
+  Description: >-
+     Checks for method calls with a space before the opening
+     parenthesis.
+  Enabled: false
+
+RequireParentheses:
+  Description: >-
+     Use parentheses in the method call to avoid confusion
+     about precedence.
+  Enabled: false
+
+ShadowingOuterLocalVariable:
+  Description: >-
+     Do not use the same name as outer local variable
+     for block arguments or block local variables.
+  Enabled: false
+
+SpaceBeforeFirstArg:
+  Description: >-
+     Put a space between a method name and the first argument
+     in a method call without parentheses.
+  Enabled: false
+
+UnderscorePrefixedVariableName:
+  Description: 'Do not use prefix `_` for a variable that is used.'
+  Enabled: false
+
+UnusedMethodArgument:
+  Description: 'Checks for unused method arguments.'
+  Enabled: false
+
+UnreachableCode:
+  Description: 'Unreachable code.'
+  Enabled: false
+
+UselessAccessModifier:
+  Description: 'Checks for useless access modifiers.'
+  Enabled: false
+
+UselessAssignment:
+  Description: 'Checks for useless assignment to a local variable.'
+  Enabled: false
+
+UselessComparison:
+  Description: 'Checks for comparison of something with itself.'
+  Enabled: false
+
+UselessElseWithoutRescue:
+  Description: 'Checks for useless `else` in `begin..end` without `rescue`.'
+  Enabled: false
+
+UselessSetterCall:
+  Description: 'Checks for useless setter call to a local variable.'
+  Enabled: false
+
+Void:
+  Description: 'Possible use of operator/literal/variable in void context.'
+  Enabled: false

data/configs/rubocop/other-metrics.yml

@@ -0,0 +1,11 @@
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/ClassLength:
+  Enabled: false
+
+Metrics/ModuleLength:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false

data/configs/rubocop/other-rails.yml

@@ -0,0 +1,33 @@
+##################### Rails ##################################
+
+ActionFilter:
+  Description: 'Enforces consistent use of action filter methods.'
+  Enabled: false
+
+DefaultScope:
+  Description: 'Checks if the argument passed to default_scope is a block.'
+  Enabled: false
+
+Delegate:
+  Description: 'Prefer delegate method for delegations.'
+  Enabled: false
+
+HasAndBelongsToMany:
+  Description: 'Prefer has_many :through to has_and_belongs_to_many.'
+  Enabled: false
+
+Output:
+  Description: 'Checks for calls to puts, print, etc.'
+  Enabled: false
+
+ReadWriteAttribute:
+  Description: 'Checks for read_attribute(:attr) and write_attribute(:attr, val).'
+  Enabled: false
+
+ScopeArgs:
+  Description: 'Checks the arguments of ActiveRecord scopes.'
+  Enabled: false
+
+Validation:
+  Description: 'Use sexy validations.'
+  Enabled: false

data/configs/rubocop/other-style.yml

@@ -0,0 +1,381 @@
+AccessModifierIndentation:
+  Description: Check indentation of private/protected visibility modifiers.
+  Enabled: false
+
+AccessorMethodName:
+  Description: Check the naming of accessor methods for get_/set_.
+  Enabled: false
+
+Alias:
+  Description: 'Use alias_method instead of alias.'
+  Enabled: false
+
+AlignHash:
+  Description: >-
+    Align the elements of a hash literal if they span more than
+    one line.
+  Enabled: false
+
+AlignParameters:
+  Description: >-
+    Align the parameters of a method call if they span more
+    than one line.
+  Enabled: false
+
+ArrayJoin:
+  Description: 'Use Array#join instead of Array#*.'
+  Enabled: false
+
+AsciiComments:
+  Description: 'Use only ascii symbols in comments.'
+  Enabled: false
+
+AsciiIdentifiers:
+  Description: 'Use only ascii symbols in identifiers.'
+  Enabled: false
+
+Attr:
+  Description: 'Checks for uses of Module#attr.'
+  Enabled: false
+
+BeginBlock:
+  Description: 'Avoid the use of BEGIN blocks.'
+  Enabled: false
+
+BlockComments:
+  Description: 'Do not use block comments.'
+  Enabled: false
+
+BlockNesting:
+  Description: 'Avoid excessive block nesting'
+  Enabled: false
+
+BlockDelimiters:
+  Description: >-
+    Avoid using {...} for multi-line blocks (multiline chaining is
+    always ugly).
+    Prefer {...} over do...end for single-line blocks.
+  Enabled: false
+
+BracesAroundHashParameters:
+  Description: 'Enforce braces style inside hash parameters.'
+  Enabled: false
+
+CaseEquality:
+  Description: 'Avoid explicit use of the case equality operator(===).'
+  Enabled: false
+
+CharacterLiteral:
+  Description: 'Checks for uses of character literals.'
+  Enabled: false
+
+ClassAndModuleChildren:
+  Description: 'Checks style of children classes and modules.'
+  Enabled: false
+
+ClassLength:
+  Description: 'Avoid classes longer than 100 lines of code.'
+  Enabled: false
+
+CollectionMethods:
+  Description: 'Preferred collection methods.'
+  Enabled: false
+
+ColonMethodCall:
+  Description: 'Do not use :: for method call.'
+  Enabled: false
+
+CommentAnnotation:
+  Description: >-
+    Checks formatting of special comments
+    (TODO, FIXME, OPTIMIZE, HACK, REVIEW).
+  Enabled: false
+
+CommentIndentation:
+  Description: 'Indentation of comments.'
+  Enabled: false
+
+CyclomaticComplexity:
+  Description: 'Avoid complex methods.'
+  Enabled: false
+
+DeprecatedHashMethods:
+  Description: 'Checks for use of deprecated Hash methods.'
+  Enabled: false
+
+Documentation:
+  Description: 'Document classes and non-namespace modules.'
+  Enabled: false
+
+DotPosition:
+  Description: 'Checks the position of the dot in multi-line method calls.'
+  Enabled: false
+
+DoubleNegation:
+  Description: 'Checks for uses of double negation (!!).'
+  Enabled: false
+
+EachWithObject:
+  Description: 'Prefer `each_with_object` over `inject` or `reduce`.'
+  Enabled: false
+
+EmptyLinesAroundAccessModifier:
+  Description: "Keep blank lines around access modifiers."
+  Enabled: false
+
+EmptyLines:
+  Description: "Keeps track of empty lines around expression bodies."
+  Enabled: false
+
+EmptyLiteral:
+  Description: 'Prefer literals to Array.new/Hash.new/String.new.'
+  Enabled: false
+
+Encoding:
+  Description: 'Use UTF-8 as the source file encoding.'
+  Enabled: false
+
+EndBlock:
+  Description: 'Avoid the use of END blocks.'
+  Enabled: false
+
+EvenOdd:
+  Description: 'Favor the use of Fixnum#even? && Fixnum#odd?'
+  Enabled: false
+
+FileName:
+  Description: 'Use snake_case for source file names.'
+  Enabled: false
+
+FlipFlop:
+  Description: 'Checks for flip flops'
+  Enabled: false
+
+FormatString:
+  Description: 'Enforce the use of Kernel#sprintf, Kernel#format or String#%.'
+  Enabled: false
+
+GlobalVars:
+  Description: 'Do not introduce global variables.'
+  Enabled: false
+
+GuardClause:
+  Description: 'Check for conditionals that can be replaced with guard clauses'
+  Enabled: false
+
+IfWithSemicolon:
+  Description: 'Never use if x; .... Use the ternary operator instead.'
+  Enabled: false
+
+IndentArray:
+  Description: >-
+    Checks the indentation of the first element in an array
+    literal.
+  Enabled: false
+
+IndentHash:
+  Description: 'Checks the indentation of the first key in a hash literal.'
+  Enabled: false
+
+Lambda:
+  Description: 'Use the new lambda literal syntax for single-line blocks.'
+  Enabled: false
+
+LambdaCall:
+  Description: 'Use lambda.call(...) instead of lambda.(...).'
+  Enabled: false
+
+LeadingCommentSpace:
+  Description: 'Comments should start with a space.'
+  Enabled: false
+
+LineEndConcatenation:
+  Description: >-
+    Use \ instead of + or << to concatenate two string literals at
+    line end.
+  Enabled: false
+
+MethodDefParentheses:
+  Description: >-
+    Checks if the method definitions have or don't have
+    parentheses.
+  Enabled: false
+
+MethodLength:
+  Description: 'Avoid methods longer than 10 lines of code.'
+  Enabled: false
+
+ModuleFunction:
+  Description: 'Checks for usage of `extend self` in modules.'
+  Enabled: false
+
+NegatedIf:
+  Description: >-
+    Favor unless over if for negative conditions
+    (or control flow or).
+  Enabled: false
+
+NegatedWhile:
+  Description: 'Favor until over while for negative conditions.'
+  Enabled: false
+
+Next:
+  Description: 'Use `next` to skip iteration instead of a condition at the end.'
+  Enabled: false
+
+NilComparison:
+  Description: 'Prefer x.nil? to x == nil.'
+  Enabled: false
+
+NonNilCheck:
+  Description: 'Checks for redundant nil checks.'
+  Enabled: false
+
+Not:
+  Description: 'Use ! instead of not.'
+  Enabled: false
+
+NumericLiterals:
+  Description: >-
+    Add underscores to large numeric literals to improve their
+    readability.
+  Enabled: false
+
+OpMethod:
+  Description: 'When defining binary operators, name the argument other.'
+  Enabled: false
+
+ParameterLists:
+  Description: 'Avoid parameter lists longer than three or four parameters.'
+  Enabled: false
+
+PercentLiteralDelimiters:
+  Description: 'Use `%`-literal delimiters consistently'
+  Enabled: false
+
+PerlBackrefs:
+  Description: 'Avoid Perl-style regex back references.'
+  Enabled: false
+
+PredicateName:
+  Description: 'Check the names of predicate methods.'
+  Enabled: false
+
+Proc:
+  Description: 'Use proc instead of Proc.new.'
+  Enabled: false
+
+RaiseArgs:
+  Description: 'Checks the arguments passed to raise/fail.'
+  Enabled: false
+
+RedundantBegin:
+  Description: "Don't use begin blocks when they are not needed."
+  Enabled: false
+
+RedundantException:
+  Description: "Checks for an obsolete RuntimeException argument in raise/fail."
+  Enabled: false
+
+RedundantSelf:
+  Description: "Don't use self where it's not needed."
+  Enabled: false
+
+RegexpLiteral:
+  Description: >-
+    Use %r for regular expressions matching more than
+    `MaxSlashes` '/' characters.
+    Use %r only for regular expressions matching more than
+    `MaxSlashes` '/' character.
+  Enabled: false
+
+RescueModifier:
+  Description: 'Avoid using rescue in its modifier form.'
+  Enabled: false
+
+SelfAssignment:
+  Description: 'Checks for places where self-assignment shorthand should have been used.'
+  Enabled: false
+
+Semicolon:
+  Description: "Don't use semicolons to terminate expressions."
+  Enabled: false
+
+SignalException:
+  Description: 'Checks for proper usage of fail and raise.'
+  Enabled: false
+
+SingleLineBlockParams:
+  Description: 'Enforces the names of some block params.'
+  Enabled: false
+
+SingleLineMethods:
+  Description: 'Avoid single-line methods.'
+  Enabled: false
+
+SingleSpaceBeforeFirstArg:
+  Description: >-
+    Checks that exactly one space is used between a method name
+    and the first argument for method calls without parentheses.
+  Enabled: false
+
+SpaceAfterControlKeyword:
+  Description: 'Use spaces after if/elsif/unless/while/until/case/when.'
+  Enabled: false
+
+SpaceAfterNot:
+  Description: Tracks redundant space after the ! operator.
+  Enabled: false
+
+SpaceBeforeComment:
+  Description: >-
+    Checks for missing space between code and a comment on the
+    same line.
+  Enabled: false
+
+SpaceInsideBlockBraces:
+  Description: >-
+    Checks that block braces have or don't have surrounding space.
+    For blocks taking parameters, checks that the left brace has
+    or doesn't have trailing space.
+  Enabled: false
+
+SpaceBeforeModifierKeyword:
+  Description: 'Put a space before the modifier keyword.'
+  Enabled: false
+
+SpaceInsideHashLiteralBraces:
+  Description: "Use spaces inside hash literal braces - or don't."
+  Enabled: false
+
+SpecialGlobalVars:
+  Description: 'Avoid Perl-style global variables.'
+  Enabled: false
+
+UnneededCapitalW:
+  Description: 'Checks for %W when interpolation is not needed.'
+  Enabled: false
+
+CommandLiteral:
+  Description: 'Checks for %x when `` would do.'
+  Enabled: false
+
+VariableInterpolation:
+  Description: >-
+    Don't interpolate global, instance and class variables
+    directly in strings.
+  Enabled: false
+
+WhenThen:
+  Description: 'Use when x then ... for one-line cases.'
+  Enabled: false
+
+WhileUntilDo:
+  Description: 'Checks for redundant do after while or until.'
+  Enabled: false
+
+WhileUntilModifier:
+  Description: >-
+    Favor modifier while/until usage when you have a
+    single-line body.
+  Enabled: false

data/lib/govuk/lint.rb

@@ -0,0 +1,9 @@
+require "govuk/lint/version"
+require "govuk/lint/cli"
+require "govuk/lint/diff"
+
+module Govuk
+  module Lint
+    CONFIG_PATH = File.expand_path("../../../configs", __FILE__)
+  end
+end

data/lib/govuk/lint/cli.rb

@@ -0,0 +1,16 @@
+require "rubocop"
+
+module Govuk
+  module Lint
+    class CLI < RuboCop::CLI
+      def run(args = ARGV)
+        args += ["--config",
+                 File.join(Govuk::Lint::CONFIG_PATH, "rubocop/all.yml")]
+
+        Diff.enable!(args) if args.include? "--diff"
+
+        super(args)
+      end
+    end
+  end
+end

data/lib/govuk/lint/diff.rb

@@ -0,0 +1,54 @@
+module Govuk
+  module Lint
+    module Diff
+      module EnabledLines
+        def enabled_line?(line_number)
+          super(line_number) &&
+            Diff.changed_lines[@processed_source.path].include?(line_number)
+        end
+      end
+
+      module TargetFinder
+        def find(args)
+          super(args).select { |f| Diff.changed_lines.keys.include? f }
+        end
+      end
+
+      def self.enable!(args)
+        args.delete("--diff")
+        @cached = true if args.delete("--cached")
+        RuboCop::Cop::Cop.prepend EnabledLines
+        RuboCop::TargetFinder.prepend TargetFinder
+      end
+
+      def self.changed_lines
+        @changed_lines ||= begin
+          changes = changed_files.map do |file|
+            next unless File.exist?(file)
+            [file, `git difftool #{commit_options}
+                    -y
+                    -x 'diff
+                      --new-line-format="%dn "
+                      --unchanged-line-format=""
+                      --changed-group-format="%>"'
+                    #{file}`.split.map(&:to_i)]
+          end
+
+          Hash[changes.compact].tap { |h| h.default = [] }
+        end
+      end
+
+    private
+      def self.changed_files
+        `git diff #{commit_options} --name-only`.
+          split.
+          select { |f| f =~ %r{.rb$} }.
+          map { |f| File.expand_path(f.chomp, "./") }
+      end
+
+      def self.commit_options
+        @cached ? "--cached origin/master" : "origin/master HEAD"
+      end
+    end
+  end
+end

data/lib/govuk/lint/version.rb

@@ -0,0 +1,5 @@
+module Govuk
+  module Lint
+    VERSION = "0.1.0"
+  end
+end

metadata

@@ -0,0 +1,112 @@
+--- !ruby/object:Gem::Specification
+name: govuk-lint
+version: !ruby/object:Gem::Version
+  version: 0.1.0
+platform: ruby
+authors:
+- Government Digital Service
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-07-07 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.9'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: gem_publisher
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.5.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.5.0
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.32'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.32'
+description: Include this in your project to easily validate it against the GDS style
+  guides
+email:
+- govuk-dev@digital.cabinet-office.gov.uk
+executables:
+- govuk-lint-ruby
+extensions: []
+extra_rdoc_files: []
+files:
+- bin/govuk-lint-ruby
+- configs/rubocop/all.yml
+- configs/rubocop/gds-ruby-styleguide.yml
+- configs/rubocop/other-lint.yml
+- configs/rubocop/other-metrics.yml
+- configs/rubocop/other-rails.yml
+- configs/rubocop/other-style.yml
+- lib/govuk/lint.rb
+- lib/govuk/lint/cli.rb
+- lib/govuk/lint/diff.rb
+- lib/govuk/lint/version.rb
+homepage: https://github.com/alphagov/govuk-lint
+licenses: []
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.4.5
+signing_key: 
+specification_version: 4
+summary: A wrapper around rubocop, configured with the GDS style guides
+test_files: []