Skip to content

Clarify LICENSE copyright holder to address legal-identity review feedback - #2

Merged
QL-UoHull merged 2 commits into
mainfrom
copilot/fix-code-review-suggestion
Jul 15, 2026
Merged

QL-UoHull merged 2 commits into
mainfrom
copilot/fix-code-review-suggestion

Conversation

Copilot AI commented Jul 15, 2026 •

Copy link
Copy Markdown
Contributor

This PR addresses a single review comment requesting a clearly identifiable copyright owner in the MIT license text. The previous holder string was ambiguous relative to legal attribution expectations.

  • What changed

    • Updated LICENSE line 3 to replace the ambiguous holder name with explicit individual copyright holders.
    • No other files or metadata were modified.
  • Applied update

- Copyright (c) 2026 QL-UoHull
+ Copyright (c) 2026 Qingde Li and Jie Tian

Copilot AI changed the title [WIP] Fix code based on review comment Clarify LICENSE copyright holder to address legal-identity review feedback Jul 15, 2026
Copilot AI requested a review from QL-UoHull July 15, 2026 11:57
@QL-UoHull
QL-UoHull marked this pull request as ready for review July 15, 2026 13:13
Copilot AI review requested due to automatic review settings July 15, 2026 13:13
@QL-UoHull
QL-UoHull merged commit 0139b01 into main Jul 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the MIT license text to make the copyright holder(s) explicitly identifiable, addressing legal-identity review feedback without changing any code or repository metadata.

Changes:

  • Replaced the ambiguous holder string in LICENSE with explicit individual names on the copyright line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants