Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mark radio.de addon as broken #4569

Open
wants to merge 1 commit into
base: matrix
Choose a base branch
from

Conversation

georgkrause
Copy link

@georgkrause georgkrause commented Sep 16, 2024

Description

As reported in #4568 the addon is not working anymore. The issue was reported to the projects issue tracker already, however its months now and there wasn't any reaction. I expect the plugin to be broken for a while, even though I actually plan to debug and eventually fix the issue. Until this is done, its only fair not to let people install the addon if its not working anyways.

I am not sure if I have to bump the version in order to mark it as broken. If there is something missing, let me know and I am happy to touch up!

Checklist:

  • My code follows the add-on rules and piracy stance of this project.
  • I have read the CONTRIBUTING document
  • Each add-on submission should be a single commit with using the following style: [plugin.video.foo] v1.0.0

@basrieter
Copy link
Contributor

@gade01 @yol @anxdpanic @MartijnKaijser Are you guys OK with this?

@georgkrause
Copy link
Author

It looks like I have to bump the version, I'd say the correct one would be v3.0.9+matrix.2?

@basrieter
Copy link
Contributor

It looks like I have to bump the version, I'd say the correct one would be v3.0.9+matrix.2?

Indeed. Or make it v3.0.10+matrix.1.

@georgkrause
Copy link
Author

But if the plugin eventually releases v3.0.10 we would be in trouble. I therefore decided for v3.0.9-matrix.2

@basrieter
Copy link
Contributor

Then they need to bump it to v3.0.11, so no worries.

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