Skip to content

chore: remove face images from tracking and gitignore image folders - #5

Closed
icecold009 with Copilot wants to merge 1 commit into
mainfrom
copilot/add-multiple-images-of-individuals
Closed

chore: remove face images from tracking and gitignore image folders#5
icecold009 with Copilot wants to merge 1 commit into
mainfrom
copilot/add-multiple-images-of-individuals

Conversation

Copilot AI commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Face images of real people were committed to the public repo. This removes them from git tracking and prevents future commits via .gitignore.

Type of Change

  • Bug fix
  • New feature
  • Refactor / code quality
  • Documentation update
  • Other: privacy / repo hygiene

Changes Made

  • Added ImagesAttendance/ and images/ to .gitignore — face data should be supplied locally, never committed
  • Ran git rm --cached -r on both folders to untrack the 30 existing image files without deleting them locally

Note: Images are removed going forward only; they remain in git history. A git filter-repo pass is needed to fully purge history.

Testing

  • Existing tests pass (pytest tests/ -v)
  • New tests added for new functionality
  • Manually tested with webcam

Checklist

  • Code follows the project style (PEP 8, max line length 120)
  • README updated if behaviour changed
  • CHANGELOG.md updated under [Unreleased]

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.

2 participants