Skip to content

CI: remove Windows from release smoke matrix - #1068

Merged
yashkrishan merged 1 commit into
mainfrom
agent/remove-windows-release-smoke
Aug 31, 2026
Merged

CI: remove Windows from release smoke matrix#1068
yashkrishan merged 1 commit into
mainfrom
agent/remove-windows-release-smoke

Conversation

@Dsantra92

Copy link
Copy Markdown
Member

Removes windows-latest from the PyPI release smoke-install matrix because the context engine local extra installs FalkorDBLite, whose redislite dependency does not support win32. Linux and macOS coverage remains across Python 3.12, 3.13, and 3.14. Validation: pre-commit check-yaml and actionlint both pass; git diff --check passes.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-31T07:43:17.630076Z 59dd0df PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: be54ad5b-f1d3-43e1-a515-b538cef3afff

📥 Commits

Reviewing files that changed from the base of the PR and between 225672b and 59dd0df.

📒 Files selected for processing (1)
  • .github/workflows/release_potpie_pypi.yml

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Tests
    • Smoke-install checks now run on Ubuntu and macOS environments.

Walkthrough

Changes

PyPI smoke-install validation

Layer / File(s) Summary
Update smoke-install platforms
.github/workflows/release_potpie_pypi.yml
The smoke-install matrix now runs on ubuntu-latest and macos-latest. It no longer runs on windows-latest.

Merge Risk: ⚪ Minimal · up to 59dd0

This localized CI change removes the unsupported Windows release smoke-install target while retaining Linux and macOS coverage; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing Windows from the release smoke-test matrix.
Description check ✅ Passed The description is directly related to the changeset. It explains why Windows was removed and identifies the remaining coverage and validation results.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)


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.

@yashkrishan
yashkrishan merged commit 8dd525a into main Aug 31, 2026
19 checks passed
@yashkrishan
yashkrishan deleted the agent/remove-windows-release-smoke branch August 31, 2026 07:46
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