search

tiliado / nuvolaplayer

Nuvola Player - Tight integration of web-based media streaming services with your desktop
https://nuvola.tiliado.eu/
BSD 2-Clause "Simplified" License
334 stars 27 forks source link

Consider renaming the "Web Worker" process to e.g. "Web Runner" #681

Closed strugee closed 2 years ago

strugee commented 3 years ago

I'm going through the docs to create a new service and stumbled when I first read this term since it's also a web platform thing. I figured out pretty fast that they weren't the same, but it might be worth renaming to avoid the ambiguity.

jiri-janousek commented 3 years ago

Hello @strugee. Thanks for your suggestion. There is also the InitWebWorker signal so it must be renamed with care. Unlikely to happen anytime soon though.

strugee commented 3 years ago

No worries! If you don't want to do this for technical/backwards compatibility reasons feel free to close - I just figured I would raise it just in case since in my experience it can be kind of hard to spot this kind of problem when you're already deep into a codebase instead of being brand new to it.

jiri-janousek commented 3 years ago

Web Runner is a much better name.

jiri-janousek commented 2 years ago

I'm closing this ticket as Nuvola Player has discontinued active development. We are going to provide only integration bug fixes on a best-effort basis until the final End-of-Life on December 31, 2022. I'm sorry that I haven't managed to resolve this ticket in time.