Closed Ephellon closed 6 years ago
Hey @Ephellon!
Wow, you made a lot of changes. Very cool. I'd love to merge the project since it wouldn't make any sense to develop separately. I will make you collaborator for this project soon.
Some things I'd like to note:
version=3.4 make release
. This will also make a separate commit with the increased version number.src/deps
to src/sites
and name the files in it index.js
instead of cs.js
(don't know what that means)?adblockplus.txt
in the repo, is this used by the code itself?const
or let
everywhere instead of var
, because those are block-scoped and cause less bugs.manifest_firefox.json
, but we'll need to find another workaround to the problem (Firefox using some keys that makes Chrome reject uploading it to the store), since I want to automate the release for both FF and Chrome.So in short, I think the way forward is to;
How does that sound to you?
Do note that I have limited time so it could take a few weeks before we can publish v4.
Sorry for not replying, but making notes of everything I've changed, should be done in a day or so
/adblockplus.txt
(won't be used)
/deps
to /sites
.../cs.*
to .../index.*
var
to let
(or const
)
for
loops/options/index.html
, /options/index.js
Web to Plex | Options | Plex Settings | Advance | Server Options | Server URL
)
localhost:32400
) for faster requests and/or security concernsconsole
with terminal
for issue #21String.prototype.toCaps
to not capitalize "on" in titlesversion=[version] make release
to build code
TL;DR "Should I make the PR (with 4K+ changes), or not?"
Cons
Pros