fix: remove lcov.info from tracked files

[AmauryVdt]

closed #125

Summary by CodeRabbit

• Chores
  • Updated ignore patterns to exclude coverage report artifacts from version control.
  • Re-affirmed handling of environment configuration files with no behavioral changes.

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated .gitignore to add lcov.info and re-applied the existing .env ignore entry (removed and re-added). No source code, exported signatures, or behavioral changes were made.

Changes

Cohort / File(s)	Summary
Repository configuration \.gitignore	Added lcov.info to the ignore list; the .env line was removed and re-added (no functional change). No other files modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I nudged lcov out of sight,
.gitignore tucked it in at night,
.env wiggled back in place,
The repo breathes with softer pace,
Hooray — a tidier codebase delight! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change—removing lcov.info from tracked files, which matches the primary objective of the PR.
Linked Issues check	✅ Passed	The PR removes lcov.info from version control and adds it to .gitignore, fully meeting the acceptance criterion in issue #125.
Out of Scope Changes check	✅ Passed	The .gitignore modifications are directly scoped to issue #125; no unrelated changes are present in the PR.
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
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.gitignore:
- Line 3: Replace the invalid gitignore pattern ".\lcov.info" with a correct
pattern: use "lcov.info" to ignore that file anywhere in the repo or use
"/lcov.info" to ignore it only at the repository root; update the entry that
currently contains ".\lcov.info" accordingly so the intended file is actually
ignored.
- Line 3: lcov.info is still tracked despite being in .gitignore; remove it from
git tracking by running either "git rm lcov.info" followed by amending the
commit, or if you need to keep the local copy use "git rm --cached lcov.info"
and then amend the commit; ensure the repository no longer lists lcov.info in
git status and that .gitignore contains the entry.

---

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cb56cd8 and 6b50cee.

📒 Files selected for processing (1)

• .gitignore

[coderabbitai]

♻️ Duplicate comments (1)

.gitignore (1)

3-3: ⚠️ Potential issue | 🟠 Major

Ignoring lcov.info is correct, but confirm it is actually untracked.

Line 3 fixes ignore behavior, but this alone does not remove an already tracked file. Please verify the PR also removes lcov.info from the git index to satisfy the acceptance criterion.

#!/bin/bash
# Verify whether lcov.info is still tracked in this branch
git ls-files | rg -n '^lcov\.info$' || echo "OK: lcov.info is not tracked"

# Optional: show whether deletion is part of current changes
git diff --name-status origin/master...HEAD | rg -n 'lcov\.info' || echo "No lcov.info entry in diff"

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.gitignore at line 3, You added lcov.info to .gitignore but must also remove
any tracked lcov.info from the git index: remove lcov.info from tracking (use
git rm --cached lcov.info and commit the removal), ensure .gitignore contains
the lcov.info entry, then verify with git ls-files that lcov.info is no longer
tracked and that the branch diff includes the removal if applicable.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In @.gitignore:
- Line 3: You added lcov.info to .gitignore but must also remove any tracked
lcov.info from the git index: remove lcov.info from tracking (use git rm
--cached lcov.info and commit the removal), ensure .gitignore contains the
lcov.info entry, then verify with git ls-files that lcov.info is no longer
tracked and that the branch diff includes the removal if applicable.

---

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6b50cee and 613b78b.

📒 Files selected for processing (2)

• .gitignore
• lcov.info

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (cb56cd8) to head (613b78b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #135   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines         3418      3418           
=========================================
  Hits          3418      3418           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.