fix: update more file headers - WPB-22589 #4072
Merged
+1,192
−1,123
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
caldrian
changed the title
caldrian
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates file headers across the codebase to comply with SwiftLint requirements. The changes standardize headers by converting block comment styles to line comments, updating copyright years to 2026, and removing trailing whitespace and unnecessary blank lines.
- Updated header comment style from
/* */block comments to//line comments - Updated copyright years from 2025, 2021, and 2016 to 2026
- Removed trailing whitespace and normalized blank lines in headers
Reviewed changes
Copilot reviewed 191 out of 195 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Localization .strings files (multiple) | Standardized headers from block to line comments, updated copyright to 2026 |
| Scripts (.sh files) | Updated copyright year from 2025 to 2026, removed trailing decorative lines |
| Settings.bundle/Root.strings | Fixed trailing whitespace on comment lines |
| Wire-iOS Accessibility.strings files | Removed extra blank lines after headers |
| WireCalling/WireMessaging Localizable.strings | Removed blank lines after header comments |
| Test resource InfoPlist.strings | Removed trailing blank lines |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Base automatically changed from
chore/update-linter-file-header-pattern
to
develop
January 6, 2026 13:02
caldrian
force-pushed
the
fix/strings-file-headers-WPB-22589
branch
from
fe58de6 to
0546dc5
Compare
jullianm
approved these changes
Jan 6, 2026
Contributor
jullianm
left a comment
jullianm
left a comment
There was a problem hiding this comment.
Oh quite a lot of files were not up to date, strange that Xcode didn't spot them. thanks for updating them!
KaterinaWire
approved these changes
Jan 6, 2026
Contributor
Test Results7 116 tests 7 089 ✅ 10m 59s ⏱️ Results for commit 0546dc5. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
This PR updates/fixes more file headers in scripts and strings files.
Checklist
[WPB-XXX].UI accessibility checklist
If your PR includes UI changes, please utilize this checklist: