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

Update sqlglot requirement from <24.2,>=23.9 to >=23.9,<25.1 #1856

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 6, 2024

Updates the requirements on sqlglot to permit the latest version.

Changelog

Sourced from sqlglot's changelog.

[v25.0.2] - 2024-06-05

✨ New Features

[v25.0.0] - 2024-06-05

🐛 Bug Fixes

🔧 Chores

[v24.1.2] - 2024-06-04

✨ New Features

[v24.1.1] - 2024-06-03

🐛 Bug Fixes

[v24.1.0] - 2024-05-30

💥 BREAKING CHANGES

✨ New Features

🐛 Bug Fixes

... (truncated)

Commits
  • 472058d Feat(bigquery): add support for GAP_FILL function
  • 8418a73 docs: update API docs, CHANGELOG.md for v25.0.1 [skip ci]
  • f6b0ec7 Revert "Refactor!!: use maybe_parse instead of convert in _binop (#3603)" (#3...
  • 262d08a docs: update API docs, CHANGELOG.md for v25.0.0 [skip ci]
  • 061be9b fix: optimize absorb_and_eliminate and remove_complements (#3595)
  • 12b6aa7 fix(snowflake): Fix COPY INTO with subquery (#3605)
  • ff7556b Refactor!!: use maybe_parse instead of convert in _binop (#3603)
  • ff55ec1 Fix(hive): generate string unit for TRUNC, parse it into TimestampTrunc too
  • 7dd244b Chore(hive): test TRUNC roundtrip
  • 5683d5f fix(hive): generate TRUNC for TimestampTrunc (#3600)
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Updates the requirements on [sqlglot](https://github.com/tobymao/sqlglot) to permit the latest version.
- [Changelog](https://github.com/tobymao/sqlglot/blob/main/CHANGELOG.md)
- [Commits](tobymao/sqlglot@v23.9.0...v25.0.2)

---
updated-dependencies:
- dependency-name: sqlglot
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot requested a review from a team June 6, 2024 15:31
@dependabot dependabot bot added dependencies Pull requests that update a dependency file python Pull requests that update Python code labels Jun 6, 2024
@dependabot dependabot bot requested a review from mwojtyczka June 6, 2024 15:31
Copy link
Collaborator

@nfx nfx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@nfx nfx merged commit f2da655 into main Jun 6, 2024
5 of 6 checks passed
@nfx nfx deleted the dependabot/pip/sqlglot-gte-23.9-and-lt-25.1 branch June 6, 2024 22:53
nfx added a commit that referenced this pull request Jun 7, 2024
* Added migration for Python linters from `ast` (standard library) to `astroid` package ([#1835](#1835)). In this release, the Python linters have been migrated from the `ast` package in the standard library to the `astroid` package, version 3.2.2 or higher, with minimal inference implementation. This change includes updates to the `pyproject.toml` file to include `astroid` as a dependency and bump the version of `pylint`. No changes have been made to user documentation, CLI commands, workflows, or tables. Testing has been conducted through the addition of unit tests. This update aims to improve the functionality and accuracy of the Python linters.
* Added workflow linter for delta live tables task ([#1825](#1825)). In this release, there are updates to the `_register_pipeline_task` method in the `jobs.py` file. The method now checks for the existence of the pipeline and its libraries, and registers each notebook or jar library found in the pipeline as a task. If the library is a Maven or file type, it will raise a `DependencyProblem` as it is not yet implemented. Additionally, new functions and tests have been added to improve the quality and functionality of the project, including a workflow linter for Delta Live Tables (DLT) tasks and a linter that checks for issues with specified DLT tasks. A new method, `test_workflow_linter_dlt_pipeline_task`, has been added to test the workflow linter for DLT tasks, verifying the correct creation and functioning of the pipeline task and checking the building of the dependency graph for the task. These changes enhance the project's ability to ensure the proper configuration and correctness of DLT tasks and prevent potential issues.
* Consistent 0-based line tracking for linters ([#1855](#1855)). 0-based line tracking has been consistently implemented for linters in various files and methods throughout the project, addressing issue [#1855](#1855). This change includes removing direct filesystem references in favor of using the Unity Catalog for table migration and format changes. It also updates comments and warnings to improve clarity and consistency. In particular, the spark-table.py file has been updated to ensure that the spark.log.level is set correctly for UC Shared Clusters, and that the Spark Driver JVM is no longer accessed directly. The new file, simple_notebook.py, demonstrates the consistent line tracking for linters across different cell types, such as Python, Markdown, SQL, Scala, Shell, Pip, and Python (with magic commands). These changes aim to improve the accuracy and reliability of linters, making the codebase more maintainable and adaptable.

Dependency updates:

 * Updated sqlglot requirement from <24.2,>=23.9 to >=23.9,<25.1 ([#1856](#1856)).
@nfx nfx mentioned this pull request Jun 7, 2024
nfx added a commit that referenced this pull request Jun 7, 2024
* Added migration for Python linters from `ast` (standard library) to
`astroid` package
([#1835](#1835)). In this
release, the Python linters have been migrated from the `ast` package in
the standard library to the `astroid` package, version 3.2.2 or higher,
with minimal inference implementation. This change includes updates to
the `pyproject.toml` file to include `astroid` as a dependency and bump
the version of `pylint`. No changes have been made to user
documentation, CLI commands, workflows, or tables. Testing has been
conducted through the addition of unit tests. This update aims to
improve the functionality and accuracy of the Python linters.
* Added workflow linter for delta live tables task
([#1825](#1825)). In this
release, there are updates to the `_register_pipeline_task` method in
the `jobs.py` file. The method now checks for the existence of the
pipeline and its libraries, and registers each notebook or jar library
found in the pipeline as a task. If the library is a Maven or file type,
it will raise a `DependencyProblem` as it is not yet implemented.
Additionally, new functions and tests have been added to improve the
quality and functionality of the project, including a workflow linter
for Delta Live Tables (DLT) tasks and a linter that checks for issues
with specified DLT tasks. A new method,
`test_workflow_linter_dlt_pipeline_task`, has been added to test the
workflow linter for DLT tasks, verifying the correct creation and
functioning of the pipeline task and checking the building of the
dependency graph for the task. These changes enhance the project's
ability to ensure the proper configuration and correctness of DLT tasks
and prevent potential issues.
* Consistent 0-based line tracking for linters
([#1855](#1855)). 0-based
line tracking has been consistently implemented for linters in various
files and methods throughout the project, addressing issue
[#1855](#1855). This change
includes removing direct filesystem references in favor of using the
Unity Catalog for table migration and format changes. It also updates
comments and warnings to improve clarity and consistency. In particular,
the spark-table.py file has been updated to ensure that the
spark.log.level is set correctly for UC Shared Clusters, and that the
Spark Driver JVM is no longer accessed directly. The new file,
simple_notebook.py, demonstrates the consistent line tracking for
linters across different cell types, such as Python, Markdown, SQL,
Scala, Shell, Pip, and Python (with magic commands). These changes aim
to improve the accuracy and reliability of linters, making the codebase
more maintainable and adaptable.

Dependency updates:

* Updated sqlglot requirement from <24.2,>=23.9 to >=23.9,<25.1
([#1856](#1856)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant