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

Fixed the indenting, typos, grammar, and single-line CSS #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jagwire16
Copy link

Thanks for your work on this project, Sean. My team uses it for almost every email we do at work. In the past, each time I downloaded the project I would fix the formatting issues. I figured it would benefit everyone if I published those changes. I also ended up fixing some consistency issues, typos, and grammar issues. Enjoy!

@jjb
Copy link

jjb commented Oct 28, 2013

@jagwire16 your commit looks nice but it mixes a few different concerns. maybe if you did 2 or three different pull requests then @seanpowell would be more likely to merge

  • typos and grammar
  • opinionated whitespace changes
  • other…?

just a thought :) @seanpowell what do you think?

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

2 participants