Skip to content

chore: ignore venv, EDA artefacts, and session notes#12

Closed
woook wants to merge 1 commit intomainfrom
chore/gitignore-venv-eda
Closed

chore: ignore venv, EDA artefacts, and session notes#12
woook wants to merge 1 commit intomainfrom
chore/gitignore-venv-eda

Conversation

@woook
Copy link
Copy Markdown
Member

@woook woook commented Apr 15, 2026

Summary

  • Add .venv/ to .gitignore (virtual environment created during EDA session)
  • Ignore eda_haemonc_v3_norm.py, haemonc_v3_norm_eda.ipynb, haemonc_v3_norm_eda.html — downstream analysis artefacts for the haemonc variant store EDA, not part of the pipeline
  • Ignore vcf_normalisation_session_*.md — session notes

Test plan

  • Verify .gitignore correctly suppresses the listed files

This change is Reviewable

Summary by CodeRabbit

  • Chores
    • Updated repository configuration to exclude development artifacts and analysis output files from version control.

Add .venv/, analysis scripts, notebooks, and session markdown files
to .gitignore — these are downstream use artefacts not part of the
normalisation pipeline itself.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3eeab507-494b-4f6a-ab34-0b2a342972b1

📥 Commits

Reviewing files that changed from the base of the PR and between f678bf5 and d745ed1.

📒 Files selected for processing (1)
  • .gitignore

📝 Walkthrough

Walkthrough

The pull request adds nine lines to .gitignore to exclude a Python virtual environment directory (.venv/) and exploratory data analysis output artifacts including Python scripts, Jupyter notebooks, HTML files, and session-specific markdown files from version control.

Changes

Cohort / File(s) Summary
Gitignore Configuration
.gitignore
Added patterns to ignore Python virtual environment (.venv/) and exploratory data analysis artifacts (EDA Python script, Jupyter notebook, HTML output, and session markdown files).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A burrow cleaned, no clutter in sight,
Virtual vines and notebooks tucked tight,
Hidden from git's all-seeing eye,
EDA sketches whispered goodbye,
Our repository hops forth, light and right!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: ignore venv, EDA artefacts, and session notes' accurately and clearly summarizes the main changes: adding gitignore patterns for a Python virtual environment, exploratory data analysis artifacts, and session notes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/gitignore-venv-eda

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.

@woook woook closed this Apr 15, 2026
@woook woook deleted the chore/gitignore-venv-eda branch April 15, 2026 14:21
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