Skip to content

Commit 733266e

Browse files
committed
rubocop and more dependency cleanup
1 parent f9949dd commit 733266e

23 files changed

+477
-112
lines changed

.rubocop.yml

Lines changed: 335 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
plugins:
2+
- rubocop-rails
3+
- rubocop-rake
4+
- rubocop-rspec
5+
- rubocop-rspec_rails
6+
17
AllCops:
28
# Exclude generated files
39
Exclude:
@@ -6,11 +12,14 @@ AllCops:
612
- 'node_modules/**/**/**/*'
713
- 'lib/**/**'
814
- 'db/*'
15+
- 'config/initializers/new_framework_defaults_7_0.rb'
916
- 'config/spring.rb'
1017
- 'config/puma.rb'
18+
- 'spec/lib/secret_spec.rb' # this will be replaced by BerkeleyLibrary::Docker
1119
inherit_mode:
1220
merge:
1321
- Exclude
22+
SuggestExtensions: false
1423

1524
# Allow one line around block body (Layout/EmptyLines will still disallow two or more)
1625
Layout/EmptyLinesAroundBlockBody:
@@ -92,6 +101,9 @@ Style/Lambda:
92101
Style/SingleLineBlockParams:
93102
Enabled: false
94103

104+
RSpec/InstanceVariable:
105+
AssignmentOnly: true
106+
95107
############################################################
96108
# Enable newer cops
97109

@@ -177,7 +189,7 @@ Lint/TripleQuotes: # (new in 1.9)
177189
Enabled: true
178190
Style/IfWithBooleanLiteralBranches: # (new in 1.9)
179191
Enabled: true
180-
Gemspec/DateAssignment: # (new in 1.10)
192+
Gemspec/DeprecatedAttributeAssignment: # (new in 1.10)
181193
Enabled: true
182194
Style/HashConversion: # (new in 1.10)
183195
Enabled: true
@@ -229,3 +241,325 @@ Style/MapToHash: # new in 1.24
229241
Enabled: true
230242
Style/NestedFileDirname: # new in 1.26
231243
Enabled: true
244+
Gemspec/AddRuntimeDependency: # new in 1.65
245+
Enabled: true
246+
Gemspec/AttributeAssignment: # new in 1.77
247+
Enabled: true
248+
Gemspec/DevelopmentDependencies: # new in 1.44
249+
Enabled: true
250+
Layout/EmptyLinesAfterModuleInclusion: # new in 1.79
251+
Enabled: true
252+
Layout/LineContinuationLeadingSpace: # new in 1.31
253+
Enabled: true
254+
Layout/LineContinuationSpacing: # new in 1.31
255+
Enabled: true
256+
Lint/ArrayLiteralInRegexp: # new in 1.71
257+
Enabled: true
258+
Lint/ConstantOverwrittenInRescue: # new in 1.31
259+
Enabled: true
260+
Lint/ConstantReassignment: # new in 1.70
261+
Enabled: true
262+
Lint/CopDirectiveSyntax: # new in 1.72
263+
Enabled: true
264+
Lint/DuplicateMagicComment: # new in 1.37
265+
Enabled: true
266+
Lint/DuplicateMatchPattern: # new in 1.50
267+
Enabled: true
268+
Lint/DuplicateSetElement: # new in 1.67
269+
Enabled: true
270+
Lint/HashNewWithKeywordArgumentsAsDefault: # new in 1.69
271+
Enabled: true
272+
Lint/ItWithoutArgumentsInBlock: # new in 1.59
273+
Enabled: true
274+
Lint/LiteralAssignmentInCondition: # new in 1.58
275+
Enabled: true
276+
Lint/MixedCaseRange: # new in 1.53
277+
Enabled: true
278+
Lint/NonAtomicFileOperation: # new in 1.31
279+
Enabled: true
280+
Lint/NumericOperationWithConstantResult: # new in 1.69
281+
Enabled: true
282+
Lint/RedundantRegexpQuantifiers: # new in 1.53
283+
Enabled: true
284+
Lint/RedundantTypeConversion: # new in 1.72
285+
Enabled: true
286+
Lint/RefinementImportMethods: # new in 1.27
287+
Enabled: true
288+
Lint/RequireRangeParentheses: # new in 1.32
289+
Enabled: true
290+
Lint/SharedMutableDefault: # new in 1.70
291+
Enabled: true
292+
Lint/SuppressedExceptionInNumberConversion: # new in 1.72
293+
Enabled: true
294+
Lint/UnescapedBracketInRegexp: # new in 1.68
295+
Enabled: true
296+
Lint/UselessConstantScoping: # new in 1.72
297+
Enabled: true
298+
Lint/UselessDefaultValueArgument: # new in 1.76
299+
Enabled: true
300+
Lint/UselessDefined: # new in 1.69
301+
Enabled: true
302+
Lint/UselessNumericOperation: # new in 1.66
303+
Enabled: true
304+
Lint/UselessOr: # new in 1.76
305+
Enabled: true
306+
Lint/UselessRescue: # new in 1.43
307+
Enabled: true
308+
Metrics/CollectionLiteralLength: # new in 1.47
309+
Enabled: true
310+
Naming/PredicateMethod: # new in 1.76
311+
Enabled: true
312+
Security/CompoundHash: # new in 1.28
313+
Enabled: true
314+
Style/AmbiguousEndlessMethodDefinition: # new in 1.68
315+
Enabled: true
316+
Style/ArrayIntersect: # new in 1.40
317+
Enabled: true
318+
Style/ArrayIntersectWithSingleElement: # new in 1.81
319+
Enabled: true
320+
Style/BitwisePredicate: # new in 1.68
321+
Enabled: true
322+
Style/CollectionQuerying: # new in 1.77
323+
Enabled: true
324+
Style/CombinableDefined: # new in 1.68
325+
Enabled: true
326+
Style/ComparableBetween: # new in 1.74
327+
Enabled: true
328+
Style/ComparableClamp: # new in 1.44
329+
Enabled: true
330+
Style/ConcatArrayLiterals: # new in 1.41
331+
Enabled: true
332+
Style/DataInheritance: # new in 1.49
333+
Enabled: true
334+
Style/DigChain: # new in 1.69
335+
Enabled: true
336+
Style/DirEmpty: # new in 1.48
337+
Enabled: true
338+
Style/EmptyHeredoc: # new in 1.32
339+
Enabled: true
340+
Style/EmptyStringInsideInterpolation: # new in 1.76
341+
Enabled: true
342+
Style/EnvHome: # new in 1.29
343+
Enabled: true
344+
Style/ExactRegexpMatch: # new in 1.51
345+
Enabled: true
346+
Style/FetchEnvVar: # new in 1.28
347+
Enabled: true
348+
Style/FileEmpty: # new in 1.48
349+
Enabled: true
350+
Style/FileNull: # new in 1.69
351+
Enabled: true
352+
Style/FileTouch: # new in 1.69
353+
Enabled: true
354+
Style/HashFetchChain: # new in 1.75
355+
Enabled: true
356+
Style/HashSlice: # new in 1.71
357+
Enabled: true
358+
Style/ItAssignment: # new in 1.70
359+
Enabled: true
360+
Style/ItBlockParameter: # new in 1.75
361+
Enabled: true
362+
Style/KeywordArgumentsMerging: # new in 1.68
363+
Enabled: true
364+
Style/MagicCommentFormat: # new in 1.35
365+
Enabled: true
366+
Style/MapCompactWithConditionalBlock: # new in 1.30
367+
Enabled: true
368+
Style/MapIntoArray: # new in 1.63
369+
Enabled: true
370+
Style/MapToSet: # new in 1.42
371+
Enabled: true
372+
Style/MinMaxComparison: # new in 1.42
373+
Enabled: true
374+
Style/ObjectThen: # new in 1.28
375+
Enabled: true
376+
Style/OperatorMethodCall: # new in 1.37
377+
Enabled: true
378+
Style/RedundantArrayConstructor: # new in 1.52
379+
Enabled: true
380+
Style/RedundantArrayFlatten: # new in 1.76
381+
Enabled: true
382+
Style/RedundantConstantBase: # new in 1.40
383+
Enabled: true
384+
Style/RedundantCurrentDirectoryInPath: # new in 1.53
385+
Enabled: true
386+
Style/RedundantDoubleSplatHashBraces: # new in 1.41
387+
Enabled: true
388+
Style/RedundantEach: # new in 1.38
389+
Enabled: true
390+
Style/RedundantFilterChain: # new in 1.52
391+
Enabled: true
392+
Style/RedundantFormat: # new in 1.72
393+
Enabled: true
394+
Style/RedundantHeredocDelimiterQuotes: # new in 1.45
395+
Enabled: true
396+
Style/RedundantInitialize: # new in 1.27
397+
Enabled: true
398+
Style/RedundantInterpolationUnfreeze: # new in 1.66
399+
Enabled: true
400+
Style/RedundantLineContinuation: # new in 1.49
401+
Enabled: true
402+
Style/RedundantRegexpArgument: # new in 1.53
403+
Enabled: true
404+
Style/RedundantRegexpConstructor: # new in 1.52
405+
Enabled: true
406+
Style/RedundantStringEscape: # new in 1.37
407+
Enabled: true
408+
Style/ReturnNilInPredicateMethodDefinition: # new in 1.53
409+
Enabled: true
410+
Style/SafeNavigationChainLength: # new in 1.68
411+
Enabled: true
412+
Style/SendWithLiteralMethodName: # new in 1.64
413+
Enabled: true
414+
Style/SingleLineDoEndBlock: # new in 1.57
415+
Enabled: true
416+
Style/SuperArguments: # new in 1.64
417+
Enabled: true
418+
Style/SuperWithArgsParentheses: # new in 1.58
419+
Enabled: true
420+
Style/YAMLFileRead: # new in 1.53
421+
Enabled: true
422+
Rails/ActionControllerFlashBeforeRender: # new in 2.16
423+
Enabled: true
424+
Rails/ActionControllerTestCase: # new in 2.14
425+
Enabled: true
426+
Rails/ActionOrder: # new in 2.17
427+
Enabled: true
428+
Rails/ActiveRecordCallbacksOrder: # new in 2.7
429+
Enabled: true
430+
Rails/ActiveSupportOnLoad: # new in 2.16
431+
Enabled: true
432+
Rails/AddColumnIndex: # new in 2.11
433+
Enabled: true
434+
Rails/AfterCommitOverride: # new in 2.8
435+
Enabled: true
436+
Rails/AttributeDefaultBlockValue: # new in 2.9
437+
Enabled: true
438+
Rails/CompactBlank: # new in 2.13
439+
Enabled: true
440+
Rails/DangerousColumnNames: # new in 2.21
441+
Enabled: true
442+
Rails/DeprecatedActiveModelErrorsMethods: # new in 2.14
443+
Enabled: true
444+
Rails/DotSeparatedKeys: # new in 2.15
445+
Enabled: true
446+
Rails/DuplicateAssociation: # new in 2.14
447+
Enabled: true
448+
Rails/DuplicateScope: # new in 2.14
449+
Enabled: true
450+
Rails/DurationArithmetic: # new in 2.13
451+
Enabled: true
452+
Rails/EagerEvaluationLogMessage: # new in 2.11
453+
Enabled: true
454+
Rails/EnumSyntax: # new in 2.26
455+
Enabled: true
456+
Rails/EnvLocal: # new in 2.22
457+
Enabled: true
458+
Rails/ExpandedDateRange: # new in 2.11
459+
Enabled: true
460+
Rails/FindById: # new in 2.7
461+
Enabled: true
462+
Rails/FindByOrAssignmentMemoization: # new in 2.33
463+
Enabled: true
464+
Rails/FreezeTime: # new in 2.16
465+
Enabled: true
466+
Rails/HttpStatusNameConsistency: # new in 2.34
467+
Enabled: true
468+
Rails/I18nLazyLookup: # new in 2.14
469+
Enabled: true
470+
Rails/I18nLocaleAssignment: # new in 2.11
471+
Enabled: true
472+
Rails/I18nLocaleTexts: # new in 2.14
473+
Enabled: true
474+
Rails/IgnoredColumnsAssignment: # new in 2.17
475+
Enabled: true
476+
Rails/Inquiry: # new in 2.7
477+
Enabled: true
478+
Rails/MailerName: # new in 2.7
479+
Enabled: true
480+
Rails/MatchRoute: # new in 2.7
481+
Enabled: true
482+
Rails/MigrationClassName: # new in 2.14
483+
Enabled: true
484+
Rails/MultipleRoutePaths: # new in 2.29
485+
Enabled: true
486+
Rails/NegateInclude: # new in 2.7
487+
Enabled: true
488+
Rails/OrderArguments: # new in 2.33
489+
Enabled: true
490+
Rails/Pluck: # new in 2.7
491+
Enabled: true
492+
Rails/PluckInWhere: # new in 2.7
493+
Enabled: true
494+
Rails/RedirectBackOrTo: # new in 2.34
495+
Enabled: true
496+
Rails/RedundantActiveRecordAllMethod: # new in 2.21
497+
Enabled: true
498+
Rails/RedundantPresenceValidationOnBelongsTo: # new in 2.13
499+
Enabled: true
500+
Rails/RedundantTravelBack: # new in 2.12
501+
Enabled: true
502+
Rails/RenderInline: # new in 2.7
503+
Enabled: true
504+
Rails/RenderPlainText: # new in 2.7
505+
Enabled: true
506+
Rails/ResponseParsedBody: # new in 2.18
507+
Enabled: true
508+
Rails/RootJoinChain: # new in 2.13
509+
Enabled: true
510+
Rails/RootPathnameMethods: # new in 2.16
511+
Enabled: true
512+
Rails/RootPublicPath: # new in 2.15
513+
Enabled: true
514+
Rails/SelectMap: # new in 2.21
515+
Enabled: true
516+
Rails/ShortI18n: # new in 2.7
517+
Enabled: true
518+
Rails/SquishedSQLHeredocs: # new in 2.8
519+
Enabled: true
520+
Rails/StripHeredoc: # new in 2.15
521+
Enabled: true
522+
Rails/StrongParametersExpect: # new in 2.29
523+
Enabled: true
524+
Rails/ThreeStateBooleanColumn: # new in 2.19
525+
Enabled: true
526+
Rails/TimeZoneAssignment: # new in 2.10
527+
Enabled: true
528+
Rails/ToFormattedS: # new in 2.15
529+
Enabled: true
530+
Rails/ToSWithArgument: # new in 2.16
531+
Enabled: true
532+
Rails/TopLevelHashWithIndifferentAccess: # new in 2.16
533+
Enabled: true
534+
Rails/TransactionExitStatement: # new in 2.14
535+
Enabled: true
536+
Rails/UnusedRenderContent: # new in 2.21
537+
Enabled: true
538+
Rails/WhereEquals: # new in 2.9
539+
Enabled: true
540+
Rails/WhereExists: # new in 2.7
541+
Enabled: true
542+
Rails/WhereMissing: # new in 2.16
543+
Enabled: true
544+
Rails/WhereNot: # new in 2.8
545+
Enabled: true
546+
Rails/WhereNotWithMultipleConditions: # new in 2.17
547+
Enabled: true
548+
Rails/WhereRange: # new in 2.25
549+
Enabled: true
550+
RSpec/IncludeExamples: # new in 3.6
551+
Enabled: true
552+
RSpec/LeakyLocalVariable: # new in 3.8
553+
Enabled: true
554+
RSpecRails/AvoidSetupHook: # new in 2.4
555+
Enabled: true
556+
RSpecRails/HaveHttpStatus: # new in 2.12
557+
Enabled: true
558+
RSpecRails/InferredSpecType: # new in 2.14
559+
Enabled: false
560+
RSpecRails/MinitestAssertions: # new in 2.17
561+
Enabled: true
562+
RSpecRails/NegationBeValid: # new in 2.23
563+
Enabled: true
564+
RSpecRails/TravelAround: # new in 2.19
565+
Enabled: true

.ruby-version

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
ruby-3.0.3
1+
ruby-3.3.10

0 commit comments

Comments
 (0)