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

Feature/clean ups #1475

Merged
merged 5 commits into from Feb 11, 2018

Conversation

Projects
None yet
3 participants
@siemensikkema
Copy link
Member

siemensikkema commented Jan 31, 2018

  • Remove extra newlines
  • Improve/fix some comments
  • Use HTTPHeaderNames in favor of Strings (depends on vapor/http#215)
  • Syntax improvements
  • Remove (near) empty files

@siemensikkema siemensikkema requested review from tanner0101 and Joannis Jan 31, 2018

@siemensikkema siemensikkema force-pushed the feature/clean-ups branch from 7b6b3a8 to 4865b41 Feb 1, 2018

@Joannis

Joannis approved these changes Feb 9, 2018

@siemensikkema siemensikkema force-pushed the feature/clean-ups branch from 4865b41 to d1f8668 Feb 11, 2018

@siemensikkema siemensikkema force-pushed the feature/clean-ups branch from d1f8668 to 3211646 Feb 11, 2018

@Joannis Joannis merged commit ed3aeea into beta Feb 11, 2018

2 checks passed

ci/circleci: linux Your tests passed on CircleCI!
Details
dependency-ci Dependencies checked
Details

@Joannis Joannis deleted the feature/clean-ups branch Feb 11, 2018

@tanner0101 tanner0101 added this to Done in Vapor 3 Feb 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment