Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

removed test images to reduce project size #105

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LTDakin
Copy link
Contributor

@LTDakin LTDakin commented Sep 16, 2024

these were used during the skeleton phase of datalab when we didn't have archive data. cleaning them up

Copy link
Contributor

@mgdaily mgdaily left a comment

Choose a reason for hiding this comment

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

Nice. These will still be present in the git history, however, so we should probably remove them all from the git history as well. Otherwise they will still be present (though, hidden) every time someone clones the repository.

GitHub has a guide about this:

https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/removing-sensitive-data-from-a-repository

@LTDakin
Copy link
Contributor Author

LTDakin commented Oct 4, 2024

Putting this on pause until all other PRs are merged (required by BFG repo cleaner to properly clean the files)

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.

3 participants