Skip to content

CI: Validate InfluxDB I/O adapter on Python 3.14#574

Merged
amotl merged 1 commit intomainfrom
python314b
Nov 18, 2025
Merged

CI: Validate InfluxDB I/O adapter on Python 3.14#574
amotl merged 1 commit intomainfrom
python314b

Conversation

@amotl
Copy link
Copy Markdown
Member

@amotl amotl commented Nov 16, 2025

About

Python 3.14 wheels for line-protocol-parser have just been published. Thanks a stack, @Penlect.

References

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 16, 2025

Warning

Rate limit exceeded

@amotl has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 2 minutes and 31 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 5d1bbe2 and 7477a76.

📒 Files selected for processing (2)
  • .github/workflows/influxdb.yml (1 hunks)
  • pyproject.toml (1 hunks)

Walkthrough

Bumps GitHub Actions Python matrix from 3.13 → 3.14 and updates optional dependency entries in pyproject.toml (switches cratedb-toolkit[io]cratedb-toolkit[io-base] and adjusts pandas version constraints).

Changes

Cohort / File(s) Change Summary
GitHub Actions workflow
.github/workflows/influxdb.yml
Updated Python version in workflow matrix from 3.133.14.
Project metadata (optional dependencies)
pyproject.toml
- optional-dependencies.influxdb: cratedb-toolkit[io]cratedb-toolkit[io-base].
- optional-dependencies.io: added pandas>=1,<2.3.
- optional-dependencies.io-base: tightened pandas constraint >=1,<2.3>=1,<2.4.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Check pyproject.toml syntax for extras/bracketed feature names and comma separation.
  • Verify workflow matrix YAML formatting and ensure no unintended changes to job steps.

Possibly related PRs

Suggested labels

dependencies

Suggested reviewers

  • surister
  • hammerhead

Poem

🐰 I hopped through YAML and toml with cheer,
Swapped Python steps and pruned versions near,
Cratedb extras and pandas bounds aligned,
A tidy patch — carrot crumbs behind! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the primary change in the changeset: updating Python version to 3.14 in the CI workflow to validate the InfluxDB I/O adapter.
Description check ✅ Passed The description is related to the changeset, explaining the context for updating to Python 3.14 (wheel availability) and referencing the dependency issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

- `influxio` covers it already, so there are no adjustments here.
- However, pandas 2.2 does not provide wheels for Python 3.14 yet, so
  let's relax dependency to pandas 2.3.
@amotl amotl marked this pull request as ready for review November 18, 2025 11:51
@amotl amotl requested review from seut and surister November 18, 2025 12:55
@amotl amotl merged commit c389a70 into main Nov 18, 2025
52 of 76 checks passed
@amotl amotl deleted the python314b branch November 18, 2025 16:03
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.

2 participants