AceFire6 / ordered-arrowverse

A listing of all shows in the Arrowverse in watch order to ensure continuity and sensible ordering for crossover episodes
https://arrowverse.info
101 stars 33 forks source link

Keep watched state 99a169a #373

Open exhuma opened 2 years ago

exhuma commented 2 years ago

This is a rebase of #325 which was closed without comment and without ever being merged. I am assuming this was because the target branch got renamed from master to main and GitHub just closed it because the target branch "disappeared"

I have based this on commit 99a169a which I assume is the last release at the timepoint when this PR was created.

I was forced to do a bit more though, so here's some detail:

Issue with dependencies

quart-minify depends on jsmin which is no longer maintained and no longer installable. After fiddling around with it for a while I gave up and had to drop the devpendency on quart-minify to even get as far as install the dependencies let alone make the thing run. This has the effect that the generated output is no longer minified though :(

Then there was an issue with markupsafe and jinja2 which forced me to limit/pin their versions. At least like this the project runs without causing any cumbersome code changes.

Issue with Python 3.10

It seems that quart is not compatible with Python 3.10. At least not in the version used in this project. So I added a VS-code dev-container to the project which runs a 3.9 environment. This has no impact whatsoever on the code. It merely provides a convenience for VS-Code developers to have a clean and reproducible environment to work in.

Dependency on Redis

The Redis dependency makes development a bit more annoying because you need a running Redis instance to start developing. I added the option to set the env-var to "None" to disable caching completely. This makes it much easier to set up a quick dev environment.

All remaning changes are related to the "keep-watched-state" feature.

If you want me to split this up in multiple PRs let me know. I could see that moving everything related to dev-environment and setup be in a separate PR. But for now, I keep it like this for housekeeping simplicity 😉

exhuma commented 1 year ago

@AceFire6 any update on this?