Skip to content

Credit Eric Litman in the README license section - #94

Merged
arjitj2 merged 1 commit into
mainfrom
arjit/readme-eric-credit
Sep 28, 2026
Merged

arjitj2 merged 1 commit into
mainfrom
arjit/readme-eric-credit

Conversation

@arjitj2

@arjitj2 arjitj2 commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Closes #93.

Name and link Eric Litman's Open Pstack adaptations in the README License section alongside Lauren Tan and Michael Denyer. The wording follows NOTICE.md. Eric's upstream LICENSE also retains Lauren's copyright notice, which is unchanged here.

Validation:

  • Documentation checker, 16 documentation tests, and whitespace checks pass.
  • Only README.md changes. The packaged plugin tree is identical to main at cfc7c8e, so no parent installation is affected.
  • Read both repositories' live LICENSE files and the current attribution notice.
  • The identical package passed the full test suite, strict typecheck, static invariants and plugin validation during the preceding release work. GitHub CI repeats the required checks for this head.

Documentation owner updated: README.md, License section.

Copilot review was requested and returned a quota-limit notice without code findings.

@arjitj2
arjitj2 requested a lite review from Copilot September 28, 2026 20:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@arjitj2

arjitj2 commented Sep 28, 2026

Copy link
Copy Markdown
Owner Author

Independent maintenance verification: PASS for 5b98e5a against cfc7c8e.

The only change is the README License paragraph. Eric's credit agrees with NOTICE.md; Lauren and Michael remain credited. The plugin tree is unchanged at 8bf32690c3e41d0b047d80a198a0d73209d2b140. Documentation and whitespace checks pass in a clean detached checkout. No findings.

@arjitj2
arjitj2 merged commit 928839a into main Sep 28, 2026
1 check passed
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.

Credit Eric Litman in the README license section

2 participants