Skip to content

Remove pyarrow Version and Update Workflow Action Version to Remove Warnings - #552

Merged
hadia206 merged 3 commits into
mainfrom
Hadia/update_pyarrow_and_action
Aug 19, 2026
Merged

Remove pyarrow Version and Update Workflow Action Version to Remove Warnings#552
hadia206 merged 3 commits into
mainfrom
Hadia/update_pyarrow_and_action

Conversation

@hadia206

Copy link
Copy Markdown
Contributor

As title.


steps:
- name: Checkout repository
uses: actions/checkout@v4

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Upgrade version to remove Action warnings

"Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-python@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/"

Comment thread pyproject.toml
# TODO: Upgrade pyarrow once the LLM package supports pyarrow>=24. Currently
# pinned to pyarrow==22 for compatibility with the LLM integration environment.
dependencies = ["pytz", "sqlglot==26.7.0", "pandas>=2.0.0", "jupyterlab", "pyarrow==22"]
dependencies = ["pytz", "sqlglot==26.7.0", "pandas>=2.0.0", "jupyterlab", "pyarrow"]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no version specified for pyarrow.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's include in the comments above here a note about pyarrow being verisonless and why

@john-sanchez31 john-sanchez31 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@knassre-bodo knassre-bodo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread pyproject.toml
# TODO: Upgrade pyarrow once the LLM package supports pyarrow>=24. Currently
# pinned to pyarrow==22 for compatibility with the LLM integration environment.
dependencies = ["pytz", "sqlglot==26.7.0", "pandas>=2.0.0", "jupyterlab", "pyarrow==22"]
dependencies = ["pytz", "sqlglot==26.7.0", "pandas>=2.0.0", "jupyterlab", "pyarrow"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's include in the comments above here a note about pyarrow being verisonless and why

@hadia206
hadia206 merged commit e5379db into main Aug 19, 2026
18 checks passed
@hadia206
hadia206 deleted the Hadia/update_pyarrow_and_action branch August 19, 2026 17:34
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 this pull request may close these issues.

3 participants