docs: add CONTRIBUTING_QUICKSTART.md for new developers#209
docs: add CONTRIBUTING_QUICKSTART.md for new developers#209swathiyara9-ai wants to merge 2 commits into
Conversation
|
@swathiyara9-ai is attempting to deploy a commit to the s3dfx-cyber's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThis PR adds ChangesContributor Quickstart Guide
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
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 `@CONTRIBUTING_QUICKSTART.md`:
- Around line 12-17: The quickstart prerequisites are missing Node.js, which
leaves the setup list incomplete. Update the prerequisites section in
CONTRIBUTING_QUICKSTART.md to include Node.js alongside Python, Docker Compose,
and Git, and add the corresponding version check in the setup command block near
the existing python version and docker compose version checks.
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: f5f03d04-2daf-40e5-a6be-9254438c387f
📒 Files selected for processing (1)
CONTRIBUTING_QUICKSTART.md
There was a problem hiding this comment.
1 issue found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="CONTRIBUTING_QUICKSTART.md">
<violation number="1" location="CONTRIBUTING_QUICKSTART.md:42">
P1: The Windows PowerShell activation command is technically incorrect for Python venv. In PowerShell, the venv activation script is `Activate.ps1`, not `activate` — the latter is a POSIX shell script that either doesn't exist on Windows or, if run, will execute in a sub-process without persisting environment changes in the current PowerShell session. This will silently fail to activate the virtual environment for Windows contributors. Consider using `.venv\Scripts\Activate.ps1` instead, and optionally noting the need to set execution policy (e.g., `Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser`) for first-time PowerShell users.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
|
@swathiyara9-ai fix bot comments and this gtg |
|
Hi! @S3DFX-CYBER I've addressed the CodeRabbit and Cubic AI review comments. The remaining Vercel workflow is awaiting maintainer approval. Please let me know if any further changes are needed. Thanks! |
Summary
Added a new
CONTRIBUTING_QUICKSTART.mdguide to help new developers quickly set up the TENET AI project and start contributing. The guide complements the existingCONTRIBUTING.mdby providing a streamlined onboarding experience.Key Changes
CONTRIBUTING_QUICKSTART.mdin the repository root.CONTRIBUTING.mdanddocs/testing.mdfor detailed documentation.Related Issue
Fixes #42
Type of Change
Screenshots / Logs (Optional)
N/A
How Has This Been Tested?
Reviewed the guide against the latest project structure,
CONTRIBUTING.md, anddocker-compose.ymlto ensure setup commands, service information, and documentation links are accurate.Checklist
Additional Notes (Optional)
This PR introduces a documentation-only quick-start guide intended to simplify onboarding for first-time contributors without replacing the existing
CONTRIBUTING.md.Summary by cubic
Adds a fast-start guide
CONTRIBUTING_QUICKSTART.mdto help new developers set up the project and run the full stack in under 10 minutes. Includes prerequisites, a 5-step setup, service URLs with health checks, a first-change walkthrough, test/quality commands (pytest,black,ruff,bandit, coverage target), troubleshooting, Windows tips, and links toCONTRIBUTING.mdanddocs/testing.md.Written for commit 9d20ccd. Summary will update on new commits.
Summary by CodeRabbit
/healthverification, and a way to stop services.POST /v1/analyze, commit).