Skip to content

Move the YouTube extension to rowkavdev/nowplaying-youtube (#136) - #426

Merged
rowkav09 merged 1 commit into
mainfrom
chore/move-youtube-extension
Sep 24, 2026
Merged

rowkav09 merged 1 commit into
mainfrom
chore/move-youtube-extension

Conversation

@rowkav09

Copy link
Copy Markdown
Member

Part of #136. Per Rowan, the browser extension lives in its own repo: https://github.com/rowkavdev/nowplaying-youtube (moved unchanged in rowkavdev/nowplaying-youtube#1, CI green there). This removes extension/youtube and its test here. The bridge, pairing token and settings API stay in this repo; the bridge notes that the extension repo mirrors its key list.

Rowan asked for the extension to live in its own repo. It moved there unchanged (rowkavdev/nowplaying-youtube#1). The bridge, pairing token and settings API stay here.
@rowkav09

Copy link
Copy Markdown
Member Author

/mira resume

@github-actions

Copy link
Copy Markdown
Contributor

/mira pause

@github-actions github-actions Bot added mira-paused Pause automatic Mira reviews on this pull request area:providers size:L labels Sep 24, 2026
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@rowkav09

Copy link
Copy Markdown
Member Author

No review from Mira (auto-paused). Self-reviewed: only removes extension/youtube and its test (the code now lives in rowkavdev/nowplaying-youtube, CI green there) and adds two pointer comments. Nothing else imports the removed files. All checks green.

@rowkav09
rowkav09 merged commit bd07846 into main Sep 24, 2026
12 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in nowplaying Sep 24, 2026
@rowkav09

Copy link
Copy Markdown
Member Author

Note: I merged this without rebasing on #429, which had landed a minute earlier (a hosted CI workflow only, no overlap with these files). CI on main (bd07846) came back green afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers mira-paused Pause automatic Mira reviews on this pull request size:L

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant