Skip to content

chore: third-party notices (superseded by #1018) - #1010

Closed
EtienneLescot wants to merge 1 commit into
mainfrom
chore/credit-recordly
Closed

EtienneLescot wants to merge 1 commit into
mainfrom
chore/credit-recordly

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

Superseded by #1018, which adds the missing third-party notices more broadly.

🤖 Generated with Claude Code

Parts of src/lib/zoomMath came from Recordly, then MIT licensed, through siddharthvaddem/openscreen#207. Its MIT notice is now reproduced in THIRD-PARTY-NOTICES.md, and the four files that still carry that code point to it.
@coderabbitai

coderabbitai Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 813ff71c-0259-496b-a32d-cc1d4dcde2fb
📥 Commits

Reviewing files that changed from the base of the PR and between 2e0733e and 192cc66.

📒 Files selected for processing (5)
  • THIRD-PARTY-NOTICES.md
  • src/lib/zoomMath/focusUtils.ts
  • src/lib/zoomMath/mathUtils.ts
  • src/lib/zoomMath/motionSmoothing.ts
  • src/lib/zoomMath/zoomRegionUtils.ts
 _____________________________________________
< Because, bugs shouldn't outnumber features. >
 ---------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@EtienneLescot
EtienneLescot deleted the chore/credit-recordly branch October 4, 2026 07:11
@EtienneLescot EtienneLescot changed the title chore: credit Recordly for the zoom math adapted from it chore: third-party notices (superseded by #1018) Oct 4, 2026
@EtienneLescot

Copy link
Copy Markdown
Collaborator Author

Closed in favor of #1018, which adds the missing third-party notices more broadly.

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.

1 participant