Skip to content

Remove deprecated APIs in Limelight code - #115

Merged
kcooney merged 5 commits into
mainfrom
kcooney/deprecated-limelight
Feb 1, 2026
Merged

Remove deprecated APIs in Limelight code#115
kcooney merged 5 commits into
mainfrom
kcooney/deprecated-limelight

Conversation

@kcooney

@kcooney kcooney commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

Also fix problems found via IntelliJ IDEA Inspection Results.

@kcooney
kcooney marked this pull request as ready for review January 11, 2026 01:12
@kcooney
kcooney force-pushed the kcooney/deprecated-limelight branch from 74e4ac9 to 9c515a8 Compare January 11, 2026 03:31
@kcooney
kcooney force-pushed the kcooney/deprecated-limelight branch from 9c515a8 to 747ed51 Compare January 15, 2026 02:27

@cuttestkittensrule cuttestkittensrule left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks good, but we should re-evaluate the test cases and make sure all the tests test something useful. I believe we still should be able to get the latency in some way, and if we can, the tests should be modified to ensure they line up

@kcooney
kcooney force-pushed the kcooney/deprecated-limelight branch 3 times, most recently from 64733b8 to 6820d51 Compare February 1, 2026 09:26

@cuttestkittensrule cuttestkittensrule left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@kcooney
kcooney force-pushed the kcooney/deprecated-limelight branch from 6820d51 to ffc1b2e Compare February 1, 2026 21:06
@kcooney
kcooney merged commit 1a33cff into main Feb 1, 2026
1 check passed
@kcooney
kcooney deleted the kcooney/deprecated-limelight branch February 1, 2026 21:14
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.

2 participants