jadjoubran / laravel5-angular-material-starter

Get started with Laravel 5.3 and AngularJS (material)
https://laravel-angular.readme.io/
MIT License
1.66k stars 400 forks source link

Update .gitignore #418

Closed nueko closed 7 years ago

nueko commented 7 years ago

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior?

Does this PR introduce a breaking change?

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

ignore .idea/

flick36 commented 7 years ago

Ummm. there's no file/folder with that name, it's generated by the IDE Intellijd or phpstorm or similar... wich not everyone uses... so i don't see the point on merging this

nueko commented 7 years ago

Yea, that's just for users who use the IDE.

GrahamCampbell commented 7 years ago

👎

GrahamCampbell commented 7 years ago

This file belongs in global .gitignore files since it's system specific.

jadjoubran commented 7 years ago

Yeah I actually learned that from @GrahamCampbell a few years ago 😄 All IDE/OS specific ignores should go into the global gitignore