Skip to content

Conversation

nikalra
Copy link
Contributor

@nikalra nikalra commented Feb 11, 2025

  • swift-lint for invoking swift-format lint with the default options
  • swift-lint-strict for invoking swift-format lint --strict

- `swift-lint` for invoking `swift-format lint` with the default options
- `swift-lint-strict` for invoking `swift-format lint --strict`
@allevato
Copy link
Member

We should probably avoid the name swift-lint to avoid confusion since there is another open-source project called SwiftLint. swift-format-lint might be better to reinforce which tool is performing the action.

@ahoppen Thoughts, since you reviewed the previous precommit check PR?

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution, @nikalra!

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Looks good to me.

@ahoppen ahoppen enabled auto-merge February 13, 2025 06:25
@ahoppen ahoppen merged commit de99480 into swiftlang:main Feb 14, 2025
19 checks passed
@nikalra nikalra deleted the pre-commit-lint branch February 14, 2025 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants