apertium / apertium-apy

📦 Apertium HTTP Server in Python
https://wiki.apertium.org/wiki/Apertium-apy
GNU General Public License v3.0
32 stars 42 forks source link

Support 'prefs' in /translateDoc #201

Closed unhammer closed 2 years ago

unhammer commented 2 years ago

Seems to work.

(Also started on translatePage, but need to sort out caching there, will do a later PR)

Cf. https://github.com/apertium/apertium-apy/issues/200

unhammer commented 2 years ago

github actions say

Run NONPAIRS=/tmp/languages pipenv run make test
flake8 *.py apertium_apy/ language_names/ tests/
/bin/sh: 1: flake8: not found
make: *** [Makefile:17: test] Error 127
Error: Process completed with exit code 2.
coveralls commented 2 years ago

Coverage Status

Coverage increased (+0.04%) to 51.427% when pulling 7dda9f65b494831ffef7331d7c256ada26d2b1cd on 200-doc+web-prefs into 1dc03451a0f5696839f0ac206906f8bbd033647b on master.

xavivars commented 2 years ago

@sushain97, it seems an issue with python matrix & flake8

sushain97 commented 2 years ago

Oh no! I'll take a look tonight. Thanks for the ping.

sushain97 commented 2 years ago

@unhammer @xavivars rebased this branch on some fixes I landed on master. Looks much better now!

unhammer commented 2 years ago

Thanks =D

(heh it always takes at least 10 commits to fix anything CI-related, doesn't it)