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

Feature: Remove .DS_STORE File and Ignore in .gitignore #224

Closed
samucodesh opened this issue Jun 12, 2024 · 6 comments · Fixed by #238
Closed

Feature: Remove .DS_STORE File and Ignore in .gitignore #224

samucodesh opened this issue Jun 12, 2024 · 6 comments · Fixed by #238
Assignees

Comments

@samucodesh
Copy link
Contributor

Suggested solution

The .DS_STORE files are generated by macOS and are not needed in the repository.

  1. Delete existing .DS_STORE file

  2. Add .DS_STORE to .gitignore:

    • Add the following line to the .gitignore file: *.DS_STORE

This will prevent future .DS_STORE files from being added to the repository, keeping it clean and efficient.

@adiati98
Copy link
Member

I agree with the proposal because we don't need the file here. However, let's also hear from @BekahHW.

If this is a go, would you like to work on it, @samucodesh?

@samucodesh
Copy link
Contributor Author

Hello @adiati98

If possible, I would like to work on it.

Also, i think it's a good idea to add: .idx/, .devcontainer/ and .vscode/ to the .gitignore file to prevent it from being added to the repository by mistake.

@BekahHW
Copy link
Member

BekahHW commented Sep 24, 2024

@samucodesh I'm sorry to have missed this. Is this something you're still interested in working on?

@samucodesh
Copy link
Contributor Author

@samucodesh I'm sorry to have missed this. Is this something you're still interested in working on?

Absolutely, I'm still very interested in working on this.

@adiati98
Copy link
Member

@samucodesh you can now claim it. 🙂

@samucodesh
Copy link
Contributor Author

.take

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants