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

Request: Updating the sqlglotrs dependency #11320

Closed
chriscc2 opened this issue Sep 6, 2024 · 4 comments · Fixed by #11331
Closed

Request: Updating the sqlglotrs dependency #11320

chriscc2 opened this issue Sep 6, 2024 · 4 comments · Fixed by #11331
Assignees

Comments

@chriscc2
Copy link
Contributor

chriscc2 commented Sep 6, 2024

Hi Datahub team,

Currently we are facing issues with adopting your library due to upstream dependencies. Currently your team is using sqlglotrs 0.2.8, which does not have a license in pypi: https://pypi.org/project/sqlglotrs/0.2.8/ and the licensing document was excluded from the library download. My team has requested that the sqlglotrs team update their download with the license agreement in version 0.2.10 and forward. I was wondering if you team could also update the dependency to pull from 0.2.10 or 0.2.11? My team is hoping to release relatively soon, but can not do so due to this. We are trying to do some legal reviews for our launch by September 14th and would love to continue using datahub.

Thanks,
Chris Chan

@shirshanka
Copy link
Contributor

Hi @chriscc2 - could you contact @hsheth2 on DataHub Slack to carry this conversation further?

@hsheth2
Copy link
Collaborator

hsheth2 commented Sep 8, 2024

This PR #11331 will bump us to sqlglotrs 0.2.12

@dbjnbnrj
Copy link

dbjnbnrj commented Sep 9, 2024

Thanks for the help @hsheth2. How can we track if the latest version with the dependency is available on pypi?

@hsheth2
Copy link
Collaborator

hsheth2 commented Sep 10, 2024

@dbjnbnrj @chriscc2 this is part of acryl-datahub v0.14.0.5

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

Successfully merging a pull request may close this issue.

4 participants