Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

Various upstream changes #2

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

eklitzke
Copy link

Hi,

I'm the author of the upstream project and think that Yelp should merge my changes to keep up to date; the three changes I pushed today (all minor) are:

  • change the servlet filter comment to be less Yelp-specific
  • add a requirements.txt file for pip
  • colorize the readme file

@@ -13,10 +13,12 @@
{% block body %}
<div class="vgap"><a href="{{uri('/')}}">Back to All Reports</a></div>
<div class="report-label">{{escape(title)}}</div>
<label for="servlet_filter">Servlet Filter: </label><input type="text" id="servlet_filter"/>
<label for="servlet_filter">{{first_column}} Filter: </label><input type="text" id="servlet_filter"/>
<!--

Choose a reason for hiding this comment

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

why not just delete this?

@dforsyth
Copy link

lgtm

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

Successfully merging this pull request may close these issues.

2 participants