Skip to content

Adjust File Headers to Use Proper Container-Compose License#17

Merged
Mcrich23 merged 4 commits intomainfrom
copilot/adjust-file-headers-license
Oct 14, 2025
Merged

Adjust File Headers to Use Proper Container-Compose License#17
Mcrich23 merged 4 commits intomainfrom
copilot/adjust-file-headers-license

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 14, 2025

  • Update license headers in all Swift files in /Sources directory
    • Replace incorrect "Apple Inc. and the container project authors" with "Morris Richman and the Container-Compose project authors"
    • Add proper Apache 2.0 license header to Application.swift which previously lacked it
    • Remove old file comment header from Application.swift per feedback
    • Keep existing file-specific comment headers (Created by, etc.) in other files
  • Verify all 26 Swift files have been updated correctly
  • Complete license header updates for the Container-Compose project

Fixes #16

Original prompt

This section details on the original issue you should resolve

<issue_title>Adjust File Headers to be Proper Licenses</issue_title>
<issue_description>The File headers in /Sources currently are a left over artifact from when this project was a Pull Request directly for Apple container. This is no longer the case and was never licensed to be Apple's code. Therefore the headers must be changed to this:

//===----------------------------------------------------------------------===//
// Copyright © 2025 Morris Richmna and the Container-Compose project authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//   https://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//===----------------------------------------------------------------------===//

It is a fairly simple change, just a matter of doing it.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #16


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>
Copilot AI changed the title [WIP] Update file headers to include proper licenses Adjust File Headers to Use Proper Container-Compose License Oct 14, 2025
Copilot AI requested a review from Mcrich23 October 14, 2025 20:11
@Mcrich23 Mcrich23 marked this pull request as ready for review October 14, 2025 20:13

//
// File.swift
// Container-Compose
Copy link
Copy Markdown
Owner

@Mcrich23 Mcrich23 Oct 14, 2025

Choose a reason for hiding this comment

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

@copilot Remove the previously existing header, replacing it with this new copyright one.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. Removed the old header comment block from Application.swift in commit f47fd4.

@Mcrich23
Copy link
Copy Markdown
Owner

@copilot For Application.swift, please remove the previously existing header, replacing it with this new copyright one.

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Oct 14, 2025

@copilot For Application.swift, please remove the previously existing header, replacing it with this new copyright one.

Done. Removed the old header comment block from Application.swift in commit f47fd4.

Copilot AI requested a review from Mcrich23 October 14, 2025 20:20
@Mcrich23
Copy link
Copy Markdown
Owner

@copilot There was a typo originally. It is Morris Richman not Morris Richmna.

…aders

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Oct 14, 2025

@copilot There was a typo originally. It is Morris Richman not Morris Richmna.

Fixed the typo in all 26 files in commit 9c8e6d5.

@Mcrich23 Mcrich23 merged commit 28a3fed into main Oct 14, 2025
@Mcrich23 Mcrich23 deleted the copilot/adjust-file-headers-license branch October 14, 2025 20:30
explicitcontextualunderstanding pushed a commit to explicitcontextualunderstanding/Container-Compose that referenced this pull request Feb 27, 2026
)

* Initial plan

* Update all Swift file headers with correct license information

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

* Remove old header from Application.swift, keep only Apache license

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

* Fix typo: Change Morris Richmna to Morris Richman in all copyright headers

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>
explicitcontextualunderstanding pushed a commit to explicitcontextualunderstanding/Container-Compose that referenced this pull request Mar 31, 2026
)

* Initial plan

* Update all Swift file headers with correct license information

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

* Remove old header from Application.swift, keep only Apache license

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

* Fix typo: Change Morris Richmna to Morris Richman in all copyright headers

Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Mcrich23 <81453549+Mcrich23@users.noreply.github.com>
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.

Adjust File Headers to be Proper Licenses

2 participants