Skip to content

Add Lint step to CI workflow#9

Merged
mattt merged 1 commit into
mainfrom
mattt/swift-format
Dec 2, 2025
Merged

Add Lint step to CI workflow#9
mattt merged 1 commit into
mainfrom
mattt/swift-format

Conversation

@mattt
Copy link
Copy Markdown
Collaborator

@mattt mattt commented Dec 2, 2025

Description

This PR updates the CI workflow to add a lint step. This will ensure consistent adherence to project code quality standards.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test coverage improvement

Changes Made

  • Adds step to run swift format lint --strict --recursive . in CI workflow

SPEC Compliance

  • This PR implements/fixes spec compliance
  • Spec section(s) affected:
  • Spec version:

Testing

  • All existing tests pass
  • Added new tests for changes
  • Tests cover edge cases and spec compliance

Pre-submission Checklist

  • My code follows the project's coding standards
  • I have run code formatting/linting tools
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally
  • I have updated documentation if needed
  • I have reviewed the TOON specification for relevant sections

Breaking Changes

  • No breaking changes
  • Breaking changes (describe migration path below)

Additional Context

@mattt mattt enabled auto-merge (squash) December 2, 2025 17:44
Copy link
Copy Markdown
Contributor

@johannschopplich johannschopplich left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@mattt mattt merged commit a33d88d into main Dec 2, 2025
3 checks passed
@mattt mattt deleted the mattt/swift-format branch December 2, 2025 21:52
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.

2 participants