Skip to content

fix: Correct modifier order in LocationService initializer#8

Merged
takeshishimada merged 1 commit intomainfrom
fix/swiftlint-modifier-order
Oct 29, 2025
Merged

fix: Correct modifier order in LocationService initializer#8
takeshishimada merged 1 commit intomainfrom
fix/swiftlint-modifier-order

Conversation

@takeshishimada
Copy link
Copy Markdown
Contributor

Summary

  • Fix SwiftLint modifier_order violation in LocationService.swift
  • Place override before nonisolated modifier

Changes

  • Examples/DemoApp/Features/LocationTracking/LocationService.swift: Changed nonisolated override to override nonisolated

Verification

  • ✅ Local SwiftLint check passes (swiftlint lint --strict)
  • ✅ Follows Swift's modifier ordering rules

Why this fix is needed

CI uses --strict mode for SwiftLint, which treats warnings as errors. The modifier_order rule requires override to come before nonisolated.

🤖 Generated with Claude Code

Fix SwiftLint modifier_order violation by placing `override` before
`nonisolated` modifier according to Swift's modifier ordering rules.

This ensures CI SwiftLint checks pass with --strict mode.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added the bug Something isn't working label Oct 29, 2025
@takeshishimada takeshishimada merged commit c18bb0f into main Oct 29, 2025
9 checks passed
@takeshishimada takeshishimada deleted the fix/swiftlint-modifier-order branch October 30, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant