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

50 suspress chmod warning #53

Merged
merged 3 commits into from
May 27, 2016

Conversation

gusdecool
Copy link
Contributor

Issue: #50

@@ -1 +1,2 @@
vendor
.idea/

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ide project folders should probably be kept in your global gitignore, FWIW

@cryptiklemur
Copy link

@schmittjoh can we get this committed and tagged? The work-around is annoying.

@aderuwe
Copy link

aderuwe commented Jul 16, 2015

@schmittjoh I really need this too (see symfony/symfony#2125 too)

@gusdecool can you please remove the .gitignore part of this PR? It has no business here

@gusdecool
Copy link
Contributor Author

@aderuwe thanks for the notice. sorry took time. was very busy this month.

PR update 👍

@vlad-ko
Copy link

vlad-ko commented Jul 31, 2015

  • 1 for this. for those using vagrant it's a big deal.

@e-moe
Copy link

e-moe commented Aug 19, 2015

👍 looking forward for merge

@aderuwe
Copy link

aderuwe commented Oct 19, 2015

@schmittjoh please could we have this merged and tagged?

@e-moe
Copy link

e-moe commented Dec 7, 2015

👍

@gusdecool
Copy link
Contributor Author

@schmittjoh any info if this PR accepted or not?

@cryptiklemur
Copy link

@schmittjoh Is there any reason this isn't being merged/tagged?

@schmittjoh schmittjoh merged commit 2cfd37d into schmittjoh:master May 27, 2016
@schmittjoh
Copy link
Owner

Thanks, it still looks a bit like a hack, but we also do not have any good alternatives.

@cryptiklemur
Copy link

Agreed. Will have to keep an eye on symfony to see if they come up with a better solution

@gusdecool gusdecool deleted the 50-suspress-chmod-warning branch May 27, 2016 07:22
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.

None yet

6 participants