search

jurismarches / luqum

A lucene query parser generating ElasticSearch queries and more !
Other
188 stars 40 forks source link

Python 2 #29

Closed fabiopedrosa closed 5 years ago

fabiopedrosa commented 6 years ago

Unfortunately, we are stuck in Py2, and lurum is only compatible with Python. The only thing making it incompatible is the yield from usage, which is very very easy to convert into supported syntax. Would that be welcomed as a PR?

alexgarel commented 6 years ago

Yes I would welcome a PR if all tests passes (add the python 2.7 environment in travis.yml).

alexgarel commented 5 years ago

No PR at the moment, so I close it.

alexgarel commented 5 years ago

Note : if some one want to propose a PR, please discuss first, for this seems very late to add all the burden of Python2 compat, while Python2 is at the end of its life.