Skip to content

Conversation

@liamnichols
Copy link
Owner

Closes #133

@liamnichols liamnichols force-pushed the ln/importsUseExplicitAccessLevel branch 3 times, most recently from 57cee7b to d7a0bc2 Compare May 30, 2025 21:11
@liamnichols liamnichols force-pushed the ln/importsUseExplicitAccessLevel branch from d7a0bc2 to eec097a Compare May 30, 2025 21:29
Comment on lines -43 to +46
// Include the access level
// Include the access level, case conversion and import preferences
components.append(configuration.accessLevel.rawValue)
components.append(configuration.convertFromSnakeCase ? "convertFromSnakeCase" : "")
components.append(configuration.importsUseExplicitAccessLevel ? "importsUseExplicitAccessLevel" : "")
Copy link
Owner Author

Choose a reason for hiding this comment

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

This is too easy to forget about 🤔

@liamnichols liamnichols marked this pull request as ready for review May 31, 2025 09:04
@liamnichols liamnichols merged commit 5cd6204 into main Jun 6, 2025
17 of 18 checks passed
@liamnichols liamnichols deleted the ln/importsUseExplicitAccessLevel branch June 6, 2025 08:49
@liamnichols liamnichols restored the ln/importsUseExplicitAccessLevel branch June 6, 2025 08:50
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.

SE-0409 • Access-level modifiers on import declarations • Support

2 participants