Skip to content

fix: remove deprecated device base class - #389

Merged
Julusian merged 1 commit into
release53from
fix/remove-deprecated-base-class
Aug 25, 2025
Merged

fix: remove deprecated device base class#389
Julusian merged 1 commit into
release53from
fix/remove-deprecated-base-class

Conversation

@Julusian

Copy link
Copy Markdown
Member

About the Contributor

This pull request is posted on behalf of Superfly

Type of Contribution

This is a: Code improvement

New Behavior

As a follow up to #375, which started on replacing the Device abstract class with an interface (to resolve stability issues with plugins), this does the work of updating the existing integrations for this deprecation and removes the old class.

Testing Instructions

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@Julusian
Julusian requested a review from a team as a code owner August 12, 2025 17:09
@codecov-commenter

codecov-commenter commented Aug 12, 2025

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 97.72727% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...line-state-resolver/src/integrations/vmix/index.ts 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Julusian
Julusian force-pushed the fix/remove-deprecated-base-class branch from e15eb90 to 501b4ab Compare August 12, 2025 17:14
@Julusian
Julusian force-pushed the fix/remove-deprecated-base-class branch from 501b4ab to 8f86527 Compare August 25, 2025 08:20
@Julusian
Julusian merged commit 81f108d into release53 Aug 25, 2025
35 of 36 checks passed
@Julusian
Julusian deleted the fix/remove-deprecated-base-class branch August 25, 2025 08:24
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