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

538 typogrify #556

Merged
merged 3 commits into from
Aug 5, 2017
Merged

538 typogrify #556

merged 3 commits into from
Aug 5, 2017

Conversation

raq929
Copy link

@raq929 raq929 commented Aug 4, 2017

Uses typogrify in all the places!

Adds Jinja2 for annoying annoying reasons: mintchaos/typogrify#44

Closes #538

@harrislapiroff harrislapiroff force-pushed the 538-typogrify branch 2 times, most recently from 8fbb0a4 to 84b62a6 Compare August 5, 2017 00:35
Copy link
Contributor

@harrislapiroff harrislapiroff left a comment

Choose a reason for hiding this comment

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

Yesssssss things are starting to look cleaner around here already! I added typogrify to a couple other locations and fixed a place where it wasn't being loaded correctly. No major changes.

@@ -30,6 +30,7 @@ gunicorn==19.7.1
html5lib==0.999999999
idna==2.5
ipaddress==1.0.18
Jinja2==2.9.6
Copy link
Contributor

Choose a reason for hiding this comment

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

So mad about this

@harrislapiroff harrislapiroff merged commit 6f2ed10 into master Aug 5, 2017
@harrislapiroff harrislapiroff deleted the 538-typogrify branch August 5, 2017 00:43
@chigby chigby mentioned this pull request Mar 9, 2023
19 tasks
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.

2 participants