updated the gitignore (little more inlusive)

akorb / SteamShutdown

Automatic shutdown after Steam download(s) has finished.

MIT License

362 stars 28 forks source link

updated the gitignore (little more inlusive) #20

Closed djreisch closed 5 years ago

djreisch commented 5 years ago

Adds basic gitignore statements which come in handy when compiling in VS

akorb commented 5 years ago

Thanks for this pull request. The /packages folder should be excluded too.

djreisch commented 5 years ago

shouldn't it be packages/? I added /packages back in so PR ready

EDIT: Added the backend forward slash so it looks nicer. Double checked it works and it does

akorb commented 5 years ago

There is a difference.

A trailing / (packages/) says "ignore directories named packages anywhere" A leading / (/packages) says "ignore files/directories named packages in the same folder as the .gitignore"

Both (/packages/) says: Ignore the directory named packages in the same folder as the .gitignore

I think we need the last one.

djreisch commented 5 years ago

I pushed the change to the branch. I had assumed you would want package folders omitted in sub-directories as well. But you're good to go, it's /packages/ which will only omit the packages folder in the root directory

akorb commented 5 years ago

I don't see a push

djreisch commented 5 years ago

Sorry about that, I pushed to master not the branch to merge. Updated now