Repository navigation
Add missing Apache 2.0 license headers to iOS sources - #424
Merged
Merged
Conversation
Adds the LBI-DHP copyright and Apache 2.0 license notice to the 40 Swift files that carried no license header. Existing Xcode preambles (filename, author, prior copyright) are preserved; the notice is inserted into the existing comment block. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Do6Ens1NwFeLxQA7DfChSB
This branch was successfully deployed
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
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.
Adds the LBI-DHP copyright and Apache 2.0 license notice to the 40 Swift files that carried no license header at all.
Existing Xcode preambles (filename, author, prior copyright lines) are preserved — the notice is inserted into the existing comment block rather than replacing it. Files with no preamble get a standalone block. The diff is purely additive (295 insertions, 0 deletions).
Note on overlap with #417
PR #417 is open for related work (removing the Commons Clause from existing headers). This PR is deliberately scoped to files that had no header at all, so the two touch disjoint sets of files.
Worth flagging separately
On
main,LICENSE.txtis plain Apache 2.0, but 424 existing source files still carry "Apache 2.0 license with Commons Clause" headers. Commons Clause is a restrictive rider that makes the terms non-open-source, so those headers contradictLICENSE.txt. #417 addresses this; it is out of scope here.Build config files (
*.gradle.kts) were left alone, matching existing convention.🤖 Generated with Claude Code
https://claude.ai/code/session_01Do6Ens1NwFeLxQA7DfChSB