Skip to content

[CI] Enable Soundness / Format check #252

Closed
@shahmishal

Description

@shahmishal
** Running swift format lint...
Sources/Algorithms/Chain.swift:93:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:95:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:123:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:125:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:133:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:136:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:189:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:193:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:208:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:222:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:230:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:241:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:251:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:253:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:255:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:258:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:271:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chain.swift:273:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Chunked.swift:456:15: warning: [AmbiguousTrailingClosureOverload] rename 'chunked(by:)' so it is no longer ambiguous when called with a trailing closure
Sources/Algorithms/Chunked.swift:489:15: note: ambiguous overload 'chunked(on:)' is here
Sources/Algorithms/EitherSequence.swift:27:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:29:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:41:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:43:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:119:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:121:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:131:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:133:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:147:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:149:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:159:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:161:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:171:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:173:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:186:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/EitherSequence.swift:188:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Joined.swift:395:15: warning: [AmbiguousTrailingClosureOverload] rename 'joined(by:)' so it is no longer ambiguous when called with a trailing closure
Sources/Algorithms/Joined.swift:409:15: note: ambiguous overload 'joined(by:)' is here
Sources/Algorithms/Intersperse.swift:104:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:105:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:106:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:108:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:138:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:140:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:156:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:158:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:160:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:161:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:212:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:213:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:217:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:218:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:219:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:220:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:224:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:225:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:239:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:240:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:241:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:242:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:246:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:247:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:251:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:252:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:265:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:267:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:391:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:392:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:402:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:403:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:404:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:405:9: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:437:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:445:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:446:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:448:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:450:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:505:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:506:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:510:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:511:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:512:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:513:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:523:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:524:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:538:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:539:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:540:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:541:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:545:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:546:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:550:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:551:7: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Intersperse.swift:570:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Reductions.swift:322:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Reductions.swift:324:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Sources/Algorithms/Reductions.swift:326:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/ChunkedTests.swift:214:7: warning: [ReplaceForEachWithForLoop] replace use of '.forEach { ... }' with for-in loop
/__w/swift-algorithms/swift-algorithms/Tests/SwiftAlgorithmsTests/KeyedTests.swift:54:58: warning: [EndOfLineComment] move end-of-line comment that exceeds the line length
Tests/SwiftAlgorithmsTests/TrimTests.swift:18:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_empty' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TrimTests.swift:24:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_nomatch' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TrimTests.swift:32:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_notailmatch' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TrimTests.swift:38:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_noheadmatch' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:77:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertEqualSequences' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TrimTests.swift:50:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_allmatch' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:89:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertUnorderedEqualSequences' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TrimTests.swift:56:9: warning: [AlwaysUseLowerCamelCase] rename the constant 'results_one' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:115:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertEqualSequences' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:135:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/TestUtilities.swift:138:10: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/TestUtilities.swift:152:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertLazySequence' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:153:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertLazyCollection' using lowerCamelCase
Tests/SwiftAlgorithmsTests/TestUtilities.swift:156:6: warning: [AlwaysUseLowerCamelCase] rename the function 'XCTAssertEqualCollections' using lowerCamelCase
Tests/SwiftAlgorithmsTests/SplitTests.swift:567:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:577:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:591:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:607:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:627:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:643:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:663:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:681:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:705:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:715:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:729:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:745:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:765:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:781:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:801:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Tests/SwiftAlgorithmsTests/SplitTests.swift:819:12: warning: [UseLetInEveryBoundCaseVariable] move this 'let' keyword inside the 'case' pattern, before each of the bound variables
Error: Process completed with exit code 123.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions