Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It relies on moz-sql-parser, which has already been archived and deprecated. #26

Closed
Taikono-Himazin opened this issue Jan 7, 2022 · 2 comments

Comments

@Taikono-Himazin
Copy link

It relies on moz-sql-parser, which has already been archived and deprecated.
https://github.com/mozilla/moz-sql-parser
Is there any attempt to make this the successor mo-sql-parsing?
https://github.com/klahnakoski/mo-sql-parsing

I looked at the source code for an hour trying to do this, but I wasn't sure. sorry.

@yashmaheshwari
Copy link
Collaborator

Hey Taikono,

Thanks for pointing that out. I wasn't aware of it -- I would read more about this new package over the weekend, and would see if we can use the forked package

The other thing I had in mind was testing with Django 4.0 as well

yashmaheshwari pushed a commit that referenced this issue Jan 24, 2022
- See Issue#26(#26) for more details
yashmaheshwari pushed a commit that referenced this issue Jan 24, 2022
- See Issue#26(#26) for more details
yashmaheshwari pushed a commit that referenced this issue Jan 24, 2022
- See Issue#26(#26) for more details
@yashmaheshwari
Copy link
Collaborator

This was pushed to master, and is fixed in the latest version (0.9)
Was getting some exceptions when trying to use for Django 4.0 - so would be doing that as part of the next PR;

Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants