search

ivelum / djangoql

Advanced search language for Django
MIT License
1k stars 88 forks source link

Consider vendoring `ply` #105

Open RaitoBezarius opened 1 year ago

RaitoBezarius commented 1 year ago

Hi there, you are using the ply dependency, as the author said in https://github.com/dabeaz/ply#important-notice---october-27-2022, he won't make any new release.

Therefore, packaging your project is difficult given this new constraint.

Can you consider vendoring this dependency like pycparser for example?

stebunovd commented 1 year ago

Hi @RaitoBezarius,

thank you for the idea and the context. I think it makes sense. We'll consider doing it in future releases.

spenserblack commented 1 year ago

Depending on how you vendor these files, you might want to mark them as vendored for GitHub.

# .gitattributes
path/to/ply/** linguist-vendored