Compilation errors probably because of the SwiftLint rules.

[TakasurAtWork]

[PerlBeforeSwine]

To work around this locally, you can temorarily disable the swiftlint build phase until the PR is merged, @TakasurAtWork.

[twostraws]

Thank you! I wonder whether we could leave the rules alone, and instead just change the constant names to be regular camel case?

[PerlBeforeSwine]

Thank you! I wonder whether we could leave the rules alone, and instead just change the constant names to be regular camel case?

@twostraws, whichever you prefer! The rule was already in place, there was just a syntax issue that I corrected in !204. If you prefer, I can update the constant names and remove the linter exception for that entirely.

[twostraws]

Let's leave the current rules alone and fix the code instead – thank you! 🙌

[PerlBeforeSwine]

Sounds good, I'll make this change to the MR.
Since the current rules file has a syntax error (two separate listings for identifier_name), I'd like to fix that as well so it's clear what the applicable rules are. Just to confirm, the preference is to remove _ from the allowed symbols in the rules file to enforce Swift conventions for naming, is that right?

[twostraws]

The preference is to write standard conventional names, like defaultLatitude.