-
-
Notifications
You must be signed in to change notification settings - Fork 134
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
chore(gitignore): remove no related project files #613
Conversation
`.idea` is editor related and `.DS_STORE` is OS related. Both of those files/folders should be inside a global gitignore file since they are not related to the project. `scripts` folder is recreate by the CLI when you bundle your application, there's no need to push compiled file so it's better to remove them by default.
Hey @RomainLanz, thanks for this PR! Unless there is a particular reason for not doing so, i'd be in favor of adding the @AStoker what do you think? |
I don't have editor configs in my repository but they are not in the project |
I'm with @JeroenVinke with adding the scripts to the list, makes sense. |
Well if you decide to keep editor/computer related files why not add |
I'm personally not opposed to it. |
Since the CLI is all about getting a working setup quickly (that you can push to github right away), I'd like to keep the editor/OS related files in the gitignore., so I agree with @AStoker. @RomainLanz could you add those editor/computer related files you mentioned, in addition to the |
Yes, so which file should I add? Should I also write a little comment on the file linking to this page if they want an advanced configuration? |
That would be good 👍 |
Okay. I have updated the file, what do you think? @JeroenVinke @AStoker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Hey everyone !
.idea
is editor related and.DS_STORE
is OS related. Both of those files/folders should be inside a global gitignore file since they are not related to the project.scripts
folder is recreate by the CLI when you bundle your application, there's no need to push compiled file so it's better to remove them by default.