Skip to content

fix: add .venv/ to gitignore templates and remove duplicate .env entry#333

Merged
notgitika merged 2 commits intomainfrom
fix/gitignore-venv
Feb 18, 2026
Merged

fix: add .venv/ to gitignore templates and remove duplicate .env entry#333
notgitika merged 2 commits intomainfrom
fix/gitignore-venv

Conversation

@notgitika
Copy link
Contributor

Description

.venv/ (the default python -m venv .venv directory) was missing from all 6 Python framework gitignore templates, causing virtual environment files (hundreds of MB of third-party packages) to be staged on git add. Also removes a duplicate .env entry in the Strands template.

Related Issue

Closes #332

Documentation PR

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

All

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@notgitika notgitika requested a review from a team February 17, 2026 23:37
@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 28.26% 1807 / 6393
🔵 Statements 27.59% 1875 / 6794
🔵 Functions 27.36% 356 / 1301
🔵 Branches 24.52% 918 / 3743
Generated in workflow #406 for commit 5e6a884 by the Vitest Coverage Report Action

Copy link
Contributor

@aidandaly24 aidandaly24 left a comment

Choose a reason for hiding this comment

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

nice

@notgitika notgitika merged commit f1c2f46 into main Feb 18, 2026
15 checks passed
@notgitika notgitika deleted the fix/gitignore-venv branch February 18, 2026 18:40
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.

Generated .gitignore missing .venv/ causes virtual environment to be committed

2 participants