JOKer

Results 28 comments of JOKer

It seems like we're still on 0.12.3. Is it a NOOP there, too? Would it make sense to expose this as a setting, so the user can choose whether they...

My own and our official ansible role still use that file: https://github.com/the-paperless-project/ansible-role-paperless/blob/master/tasks/main.yml#L80 EDIT: Might be possible to fix this with https://github.com/pypa/pipenv/issues/363#issuecomment-421310544 or by generating a `requirements.txt` like this https://github.com/pypa/pipenv/blob/master/docs/advanced.rst#-generating-a-requirementstxt

Took the liberty to fix the pycodestyle errors shown in travis. It was just line length. That whole matching section gives me shivers, though. Every if seems written in a...

We still have test failures with these patches. Going to take a look.

`_split_match()` returns spaces replaced with `\s+`. Also `re.escape()` escapes spaces, so the whole `_split_match()` magic is a little hard to fix. ``` >>> print(re.escape('alpha charlie gamma')) alpha\ charlie\ gamma ```

```diff diff --git a/src/documents/models.py b/src/documents/models.py index 7d12f91..f368f31 100644 --- a/src/documents/models.py +++ b/src/documents/models.py @@ -98,15 +98,14 @@ class MatchingModel(models.Model): if self.matching_algorithm == self.MATCH_ALL: for word in self._split_match(): search_result = re.search( -...

I'd prefer something like this. Opinions? ```python findterms = re.compile(r'"([^"]+)"|(\S+)').findall normspace = re.compile(r"\s+").sub # find all terms and replace multiple spaces with a single one terms = [normspace(" ", (t[0]...

I only know how to do this for deployments using uWSGI. This is part of my nginx config for another project: ``` location /blog/ { uwsgi_pass 127.0.0.1:8780; include uwsgi_params; uwsgi_param...