From 8aa697363b66e537a3c9689b7dbe18a9f3872dd2 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Fri, 27 Oct 2017 09:01:38 +0100 Subject: test.flac: add flac fixture. --- Library/Homebrew/test/support/fixtures/test.flac | Bin 0 -> 98 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 Library/Homebrew/test/support/fixtures/test.flac (limited to 'Library') diff --git a/Library/Homebrew/test/support/fixtures/test.flac b/Library/Homebrew/test/support/fixtures/test.flac new file mode 100644 index 000000000..253f14a71 Binary files /dev/null and b/Library/Homebrew/test/support/fixtures/test.flac differ -- cgit v1.2.3 From 5a8d2352c2012504544cda53eab55eec324f4a82 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Thu, 26 Oct 2017 16:06:29 +0100 Subject: rubocop.yml: reduce tap maximum line length. --- Library/.rubocop.yml | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'Library') diff --git a/Library/.rubocop.yml b/Library/.rubocop.yml index 2580668b4..051a57f93 100644 --- a/Library/.rubocop.yml +++ b/Library/.rubocop.yml @@ -58,7 +58,7 @@ Lint/EndAlignment: Lint/ParenthesesAsGroupedExpression: Enabled: false -# TODO: try to bring down all metrics maximums +# most metrics don't make sense to apply for formulae/taps Metrics/AbcSize: Enabled: false @@ -72,9 +72,10 @@ Metrics/CyclomaticComplexity: Enabled: false Metrics/LineLength: - Max: 324 - # ignore manpage comments - IgnoredPatterns: ['#: '] + # 124 is the goal as GitHub diff UI wraps beyond that + Max: 190 + # ignore manpage comments and long single-line strings + IgnoredPatterns: ['#: ', ' url "', ' mirror "', ' plist_options :'] Metrics/MethodLength: Enabled: false -- cgit v1.2.3 From 47bfad64b05346fc6ba2e0c57cc6c1e2318c109c Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Fri, 27 Oct 2017 09:34:38 +0100 Subject: rubocop.yml: re-add Homebrew metrics maximums. --- Library/Homebrew/.rubocop.yml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) (limited to 'Library') diff --git a/Library/Homebrew/.rubocop.yml b/Library/Homebrew/.rubocop.yml index 0e1fb2d04..354867738 100644 --- a/Library/Homebrew/.rubocop.yml +++ b/Library/Homebrew/.rubocop.yml @@ -25,6 +25,10 @@ Lint/NestedMethodDefinition: Lint/ParenthesesAsGroupedExpression: Enabled: true +# TODO: try to bring down all metrics maximums +Metrics/AbcSize: + Max: 250 + Metrics/BlockLength: Max: 1250 @@ -34,12 +38,24 @@ Metrics/BlockNesting: Metrics/ClassLength: Max: 1226 +Metrics/CyclomaticComplexity: + Max: 75 + Metrics/LineLength: + # 124 is the goal as GitHub diff UI wraps beyond that Max: 244 + # ignore manpage comments + IgnoredPatterns: ['#: '] Metrics/MethodLength: Max: 195 +Metrics/ModuleLength: + Max: 222 + +Metrics/PerceivedComplexity: + Max: 100 + # we won't change backward compatible method names Naming/MethodName: Exclude: -- cgit v1.2.3