Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Ignore ALL .log files at gitignore #523

Closed
wants to merge 6 commits into from
Closed

Conversation

simison
Copy link
Member

@simison simison commented Apr 19, 2015

Smart practise.

E.g. when app is served with Phusion Passenger or some other app server, you might end up having some log files at the root folder.

Smart practise. 

E.g. when app is served with Phusion Passenger or some other app server, you might end up having some log files at the root folder.
@ilanbiala ilanbiala added this to the 0.4.0 milestone Apr 19, 2015
@ilanbiala ilanbiala self-assigned this Apr 19, 2015
@ilanbiala
Copy link
Member

@lirantal no issues with this, right?

@lirantal
Copy link
Member

yeah that's cool.

@ilanbiala
Copy link
Member

@simison can you rebase and then I'll merge?

vepasto and others added 2 commits May 7, 2015 14:26
Email address should not be shown to client
Hide email address when reseting password
@ilanbiala
Copy link
Member

@simison any issues rebasing?

ilanbiala and others added 3 commits May 17, 2015 11:05
Add fonts and svgs to gzip compression pattern
Smart practise. 

E.g. when app is served with Phusion Passenger or some other app server, you might end up having some log files at the root folder.
@simison
Copy link
Member Author

simison commented May 17, 2015

Sorry man, postponed this earlier with no time and then just forgot about this.

I'm no git ninja, did that now went as it shoulda?

(in any case feel free to just add that change yourself, too).

@ilanbiala
Copy link
Member

I think some things got copied from another commit, so I'm just going to commit it myself.

ilanbiala added a commit that referenced this pull request May 18, 2015
@ilanbiala ilanbiala closed this May 18, 2015
@simison
Copy link
Member Author

simison commented May 18, 2015

👍

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

Successfully merging this pull request may close these issues.

4 participants