Skip to content

Remove Swift Lint #120

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 17, 2025
Merged

Remove Swift Lint #120

merged 1 commit into from
Jun 17, 2025

Conversation

goergisn
Copy link
Contributor

Removing Swift Lint to mitigate build issues

Copy link

Copy link

⚠️ 2 public changes detected ⚠️

Comparing remove-swift-lint to main

🔀1 Modification
1 Removal

Package.swift

.dependencies

❌ Removed

.package(url: "https://github.com/SimplyDanny/SwiftLintPlugins", "0.59.1"..<"1.0.0")

.targets

🔀 Modified

// From
.executableTarget(name: "public-api-diff", dependencies: [.target(name: "PADProjectBuilder"), .target(name: "PADSwiftInterfaceDiff"), .target(name: "PADOutputGenerator"), .target(name: "PADPackageFileAnalyzer"), .target(name: "PADSwiftInterfaceFileLocator"), .product(name: "ArgumentParser", ...), .product(name: "SwiftLintBuildToolPlugin", ...)], path: "Sources/ExecutableTargets/CommandLineTool")

// To
.executableTarget(name: "public-api-diff", dependencies: [.target(name: "PADProjectBuilder"), .target(name: "PADSwiftInterfaceDiff"), .target(name: "PADOutputGenerator"), .target(name: "PADPackageFileAnalyzer"), .target(name: "PADSwiftInterfaceFileLocator"), .product(name: "ArgumentParser", ...)], path: "Sources/ExecutableTargets/CommandLineTool")

/**
Changes:
- Removed dependency .product(name: "SwiftLintBuildToolPlugin", ...)
*/

Analyzed targets: FileHandlingModule, PADCore, PADLogging, PADOutputGenerator, PADPackageFileAnalyzer, PADProjectBuilder, PADSwiftInterfaceDiff, PADSwiftInterfaceFileLocator, ShellModule, SwiftPackageFileHelperModule

@goergisn goergisn merged commit 61d3be1 into main Jun 17, 2025
8 checks passed
@goergisn goergisn deleted the remove-swift-lint branch June 17, 2025 12:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant