Skip to content

Conversation

@fltb
Copy link

@fltb fltb commented Jul 18, 2025

This PR deletes untracked files that should not be committed to the repository, including:

  • node_modules/: dependency directory, should be installed via package manager
  • .DS_Store: macOS system file, unnecessary for version control

Also added a .gitignore file to prevent these from being committed in the future.

@4k1k0
Copy link

4k1k0 commented Jul 19, 2025

did you just clone the repo and rm -rf node_modules ? This will not work. @Jackywine needs to rewrite all the git history 😆

@fltb
Copy link
Author

fltb commented Jul 20, 2025

That’s not something I have the authority to do.
Other people can also use the --depth parameter of git clone.

@just-ign
Copy link

@4k1k0 , I don't think the git history needs to be written. SImply deleting the file and then adding it to the .gitignore (Like in this PR) is more than enough. That's how I usually go about it..

@plsgivemeachane
Copy link

Pushing node_modules in repo is wild🙏🏻

@Jackywine
Copy link
Owner

不好意思,我并不会写代码,只是一路让 AI 继续,给大家造成了困扰
Sorry, I can't write code, I just let the AI continue all the way, causing trouble for everyone

@array-in-a-matrix
Copy link

array-in-a-matrix commented Jul 21, 2025

https://stackoverflow.com/questions/10067848/remove-folder-and-its-contents-from-git-githubs-history/61544937#61544937

TL;DR:
Install git-filter-repo then run git-filter-repo --path node_modules --invert-paths.

This PR does not solve the issue as git clone ... will still include the node_modules and DS_Store in the git history.

@plsgivemeachane
Copy link

不好意思,我并不会写代码,只是一路让 AI 继续,给大家造成了困扰 Sorry, I can't write code, I just let the AI continue all the way, causing trouble for everyone

Its fine. The fact that you're open to letting others contribute and improve the code is already super helpful!

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.

6 participants