[6.0] Respect module name argument order precedence

swiftlang / sourcekit-lsp

Language Server Protocol implementation for Swift and C-based languages

Apache License 2.0

3.3k stars 273 forks source link

Closed plemarquand closed 2 months ago

plemarquand commented 3 months ago

Explanation: When the same compiler flag is specified multiple times the last one found should be used.
Scope: workspace/tests and textDocument/tests requests
Original PRs: https://github.com/swiftlang/sourcekit-lsp/pull/1530
Risk: Low, this just brings in line the moduleName function with how SwiftPM parses arguments.
Testing: Added test case
Reviewers: @ahoppen in https://github.com/swiftlang/sourcekit-lsp/pull/1549

ahoppen commented 2 months ago

@swift-ci Please test