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

Removes booleans class and type #25

Merged
merged 1 commit into from
Jan 21, 2016
Merged

Conversation

jwg2s
Copy link
Contributor

@jwg2s jwg2s commented Jan 21, 2016

Developer Checklist

  • I considered alternative approaches and chose the simplest one.
  • I wrote unit/acceptance specs for these changes that test all possible conditions (success, failure, bad input, etc).
  • I wrote unit/acceptance specs for code/functionality these changes may have affected.
  • I can't think of any other code/functionality this may have affected that I didn't test or account for.
  • I've ensured any necessary data migrations have been run.
  • I've ensured other blocking code has been deployed (e.g. if part of a larger feature).
  • We are able to track any changes to application performance or utilization introduced by this PR.

Deploying is blocked by:

  • Nothing :shipit:

shaunr0b pushed a commit that referenced this pull request Jan 21, 2016
@shaunr0b shaunr0b merged commit fe9228e into develop Jan 21, 2016
@shaunr0b shaunr0b deleted the feature/remove-boolean-class branch January 21, 2016 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants