Skip to content

fix: correct workspace count in CONTRIBUTING.md and Python version in publish.yml - #6714

Open
haroldfabla2-hue wants to merge 1 commit into
crewAIInc:mainfrom
haroldfabla2-hue:fix/doc-6664-6662
Open

fix: correct workspace count in CONTRIBUTING.md and Python version in publish.yml#6714
haroldfabla2-hue wants to merge 1 commit into
crewAIInc:mainfrom
haroldfabla2-hue:fix/doc-6664-6662

Conversation

@haroldfabla2-hue

Copy link
Copy Markdown

Summary

Two small documentation/config fixes reported in issues #6664 and #6662:

Fix 1: Update workspace package count (#6664)

  • File: .github/CONTRIBUTING.md
  • Change: four packagessix packages
  • Reason: The pyproject.toml defines 6 workspace members under lib/:
[tool.uv.workspace]
members = [
    "lib/crewai",
    "lib/crewai-tools",
    "lib/devtools",
    "lib/crewai-files",
    "lib/cli",
    "lib/crewai-core",
]

Fix 2: Align publish.yml Python version (#6662)

  • File: .github/workflows/publish.yml (lines 34 and 69)
  • Change: python-version: "3.12"python-version: "3.13"
  • Reason: Aligns with .python-version (3.13) and removes inconsistency

Checklist

  • Follows contributing guidelines
  • References related issues

- Fixes crewAIInc#6664: Update CONTRIBUTING.md to say 'six packages' instead of 'four'
  (pyproject.toml defines 6 workspace members: crewai, crewai-tools, devtools,
  crewai-files, cli, crewai-core)

- Fixes crewAIInc#6662: Align publish.yml python-version with .python-version (3.13)
  to resolve inconsistency across .python-version (3.13), publish.yml (was 3.12),
  and CONTRIBUTING.md
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The repository documentation now lists six lib/ workspace packages. The PyPI publishing workflow uses Python 3.13 in both build and publish jobs.

Changes

Repository documentation

Layer / File(s) Summary
Workspace package count
.github/CONTRIBUTING.md
The repository structure documentation now states that the lib/ uv workspace contains six packages instead of four.

Publish workflow runtime

Layer / File(s) Summary
Python 3.13 publish jobs
.github/workflows/publish.yml
The build and publish jobs now configure Python 3.13 instead of Python 3.12.

Suggested reviewers: lorenzejay

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes both changes: the workspace count fix and the Python version update.
Description check ✅ Passed The description clearly matches the documented documentation and workflow changes in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Tools execution failed with the following error:

Failed to run tools: 14 UNAVAILABLE: read ECONNRESET


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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/CONTRIBUTING.md:
- Line 32: Add the two missing workspace packages to the package table in
CONTRIBUTING.md so all six members under lib/ are listed, keeping the existing
table structure and descriptions consistent.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: cb692f9b-4961-4b8d-b3a5-07491197bd06

📥 Commits

Reviewing files that changed from the base of the PR and between f15844b and 142d589.

📒 Files selected for processing (2)
  • .github/CONTRIBUTING.md
  • .github/workflows/publish.yml

Comment thread .github/CONTRIBUTING.md
## Repository Structure

This is a uv workspace with four packages under `lib/`:
This is a uv workspace with six packages under `lib/`:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

List all six workspace packages in the table.

The heading now claims six packages, but the table below still documents only four. Add the two missing workspace members so contributors can discover the complete workspace structure.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/CONTRIBUTING.md at line 32, Add the two missing workspace packages
to the package table in CONTRIBUTING.md so all six members under lib/ are
listed, keeping the existing table structure and descriptions consistent.

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.

1 participant