summaryrefslogtreecommitdiffstatshomepage
Commit message (Collapse)AuthorAgeLines
* build/deps: Update python dependenciesWolfgang Müller5 days-90/+90
|
* release: Version 0.3.00.3.0Wolfgang Müller12 days-6/+36
|
* backend/lint: Do not shadow certain builtinsWolfgang Müller12 days-56/+60
| | | | | | | | | | | | This commit enables ruff's flake8-builtin linter that emits warnings when builtin functions are shadowed. This is useful for builtins like "dict", "list", or "str" which we use often. Given the nature of this program we historically rely a lot on the usage of "id", "hash", and "filter" as variable names which also shadow Python builtins. For now let's ignore those, we have not used any of them in our code and the impact to the codebase would be considerable. This might be revisited in the future.
* backend/api: Remove superfluous scalar definitionWolfgang Müller12 days-5/+0
| | | | | Sadly we couldn't find out what this was meant to do, but we believe it was added in error. All tests pass with this removed, so we can drop it.
* backend/plugins: Have E-Hentai raise an error from status_code earlyWolfgang Müller12 days-19/+19
| | | | This makes the code clearer and saves a whole indentation level.
* backend/plugins: Catch E-Hentai errors only for relevant linesWolfgang Müller12 days-11/+11
|
* backend/plugins: Throw error if E-Hentai response is missing 'gmetadata'Wolfgang Müller12 days-0/+18
| | | | | | | It might be that we get a valid (maybe empty) response from the API, in which case we do not want to simply crash because we expect the 'gmetadata' field in the response. Instead, throw a proper ScrapeError for it.
* backend/tests: Add tests for the E-Hentai APIWolfgang Müller12 days-1/+159
|
* backend/plugins: Remove stray apostropheWolfgang Müller12 days-1/+1
|
* backend/tests: Add tests for the anchira scraperWolfgang Müller12 days-0/+107
|
* backend/tests: Remove unneeded parameter in test_gallery_dlWolfgang Müller12 days-1/+1
|
* backend/plugins: Have exhentai assume no censorship for non-hWolfgang Müller12 days-2/+8
| | | | | | Non-H usually has nothing to censor, so this should be a safe default. We have not come across anything where this would have been a false positive.
* backend/plugins: Use language parser from scraper utilsWolfgang Müller13 days-32/+5
| | | | | | Now that we have this in our utility suite, we can make use of it in the built-in scraper plugins. This increases coverage and removes a lot of duplicate code.
* backend/tests: Add tests for gallery_dl scrapersWolfgang Müller13 days-1/+647
|
* backend/scraper: Have collect() ignore None resultsWolfgang Müller13 days-3/+21
| | | | | | If a parser function returned None we yield it regardless, even though it won't have any impact further down the line. Instead clean up the collect() stream as early as possible.
* backend/tests: Add test for open_archive_fileWolfgang Müller13 days-2/+41
|
* backend/plugins: Use "no cover" pragma for consistencyWolfgang Müller13 days-1/+1
|
* backend/plugins: Fix MangaDex scraper title formattingWolfgang Müller13 days-1/+1
|
* backend/scraper: Add parser methods for LanguageWolfgang Müller13 days-0/+65
| | | | | | We can expect a number of scraper sources to either give languages as ISO 639-3 or as their English name, so it makes sense to implement a simple parser method on our side.
* build: Mark coverage as PHONY targetWolfgang Müller13 days-1/+1
| | | | | Otherwise make will refuse to regenerate coverage information if the coverage/ directory already exists.
* build/deps: Remove black as a development dependencyWolfgang Müller13 days-85/+2
| | | | We can do formatting now with ruff(1).
* backend/lint: Format overlong lineWolfgang Müller13 days-1/+3
|
* build/deps: Update sphinx to 8.1.3 and sphinx-issues to 5.0.0Wolfgang Müller13 days-23/+23
|
* build: Remove obsolete pytest ignore in MakefileWolfgang Müller13 days-1/+1
| | | | | The DeprecationWarnings we ignored here were not caused by our own code. It seems whatever upstream project was affected has since been fixed.
* build/deps: Update strawberry-graphql to 0.248.1Wolfgang Müller13 days-7/+6
|
* build/deps: Update uvicorn to 0.32.0Wolfgang Müller13 days-5/+5
|
* build/deps: Update starlette to 0.41.2Wolfgang Müller13 days-5/+5
|
* build/deps: Update pytest-cov to 6.0.0Wolfgang Müller13 days-7/+7
|
* backend/lint: Ignore B027 in api/inputs.pyWolfgang Müller13 days-1/+1
| | | | | | | Even though our base class here is abstract, this method is not, so we can ignore B027 [1]. [1] https://docs.astral.sh/ruff/rules/empty-method-without-abstract-decorator/
* backend/lint: Properly chain exceptionsWolfgang Müller13 days-6/+6
| | | | | | This fixes flake8-bugbear's B904 [1]. [1] https://docs.astral.sh/ruff/rules/raise-without-from-inside-except/
* backend/plugins: Have anchira scraper use parse_dict from scraper utilsWolfgang Müller13 days-11/+2
| | | | | | This cuts down on code duplication and also fixes B023 [1]. [1] https://docs.astral.sh/ruff/rules/function-uses-loop-variable/#function-uses-loop-variable-b023
* backend/lint: Ignore B007Wolfgang Müller13 days-1/+1
| | | | | | | | B007 is "unused-loop-control-variable", which is only enabled by default because an unused variable in a loop statement "can confuse readers". The only couple of places that happens in this codebase is when unpacking os.walk, which really shouldn't be a problem.
* backend/lint: Stop using mutable objects as function argument defaultsWolfgang Müller13 days-3/+12
| | | | See https://docs.astral.sh/ruff/rules/mutable-argument-default/
* backend/lint: Implement pyupgrade suggestionsWolfgang Müller13 days-1/+1
|
* backend/lint: Implement flake8-simplify suggestionsWolfgang Müller13 days-32/+29
|
* backend/lint: Ignore SIM102 and SIM108Wolfgang Müller13 days-0/+2
| | | | | | | | | | | SIM102 is "collapsible-if", which we always follow except a few specific cases where we believe nested ifs are easier to read. Since we'd have to manually ignore all those instances, disable the whole rule instead. SIM108 is "if-else-block-instead-of-if-exp" which suggests the use of the ternary operator instead of an if/else block. We generally dislike how the ternary operator is less readable than a simple if/else block, so we disable this rule.
* backend/scraper: Bind loop variables correctlyWolfgang Müller13 days-2/+2
| | | | | This was uncovered by bugbear, but did not seem to have tripped our test. Fix it anyway.
* backend/lint: Add some more useful lintersWolfgang Müller13 days-2/+12
| | | | These should come in handy in the general case, especially bugbear.
* backend/lint: Fix import formattingWolfgang Müller13 days-17/+37
|
* build/deps: Update ruff to 0.7.3Wolfgang Müller13 days-21/+21
|
* build/deps: Update pillow to 11.0.0Wolfgang Müller13 days-85/+80
|
* build/deps: Update python dependenciesWolfgang Müller13 days-711/+784
|
* build/deps: Lock dependencies at latest versionsWolfgang Müller13 days-42/+42
| | | | | This does not include the update to Svelte 5 which will be made separately in the future.
* build/deps: Update selve-toast to 0.9.6Wolfgang Müller13 days-2/+2
|
* build/deps: Update prettier-plugin-tailwindcss to 0.6.8Wolfgang Müller13 days-5/+9
|
* build/deps: Update date-fns to 4.1.0Wolfgang Müller13 days-5/+5
|
* build/deps: Update npm-check-updates to 17.1.11Wolfgang Müller13 days-3440/+23
|
* build/deps: Update eslintWolfgang Müller13 days-266/+362
|
* build/deps: Update JS dependenciesWolfgang Müller13 days-1480/+1102
|
* release: Version 0.2.00.2.0Wolfgang Müller2024-07-22-6/+20
|