SwiftLint icon indicating copy to clipboard operation
SwiftLint copied to clipboard

Windows Port

Open compnerd opened this issue 2 years ago • 3 comments

This is currently not possible to build on Windows. It requires that SourceKitten changes are merged (https://github.com/jpsim/SourceKitten/pull/769) and some changes to the toolchain that are still in the process of getting merged upstream (https://github.com/apple/swift-package-manager/pull/6573) which depends on a toolchain that is not yet tagged.

It is not possible to build the tests on Windows due to the use of the executable target as a dependency for the tests which results in multiple definitions of main.

This will require a fix for libdispatch (apple/swift-corelibs-libdispatch#791) as well. With that, it seems that I can now swiftlint on Windows as well!

compnerd avatar May 22 '23 21:05 compnerd

2 Warnings
:warning: If this is a user-facing change, please include a CHANGELOG entry to credit yourself!
You can find it at CHANGELOG.md.
:warning: This PR may need tests.
1 Message
:book: Skipping OSSCheck because SwiftLint hasn't changed compared to 'main'

Here's an example of your CHANGELOG entry:

* Windows Port.  
  [compnerd](https://github.com/compnerd)
  [#issue_number](https://github.com/realm/SwiftLint/issues/issue_number)

note: There are two invisible spaces after the entry's text.

Generated by :no_entry_sign: Danger

SwiftLintBot avatar May 22 '23 21:05 SwiftLintBot

The only thing needed is to get the SourceKitten changes in (but haven't been able to get response from @jpsim to get those resolved and merged).

compnerd avatar Jan 08 '24 18:01 compnerd

The only thing needed is to get the SourceKitten changes in (but haven't been able to get response from @jpsim to get those resolved and merged).

Last I checked the windows PRs for some SwiftLint dependencies failed to build or failed tests for macOS or Linux. I’m happy to review again when those failures are addressed.

jpsim avatar Jan 08 '24 22:01 jpsim