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

Raise error when checking a size of a json ruby value of nil #82

Merged
merged 2 commits into from
Oct 10, 2014

Conversation

eluns
Copy link
Contributor

@eluns eluns commented Oct 6, 2014

Hi Steve,
Please let me know what you think about the tests.
Thanks,
Beth

@laserlemon
Copy link
Contributor

Travis is failing with your changes. I'm not available to dig into it but please take a look. I'll also leave some feedback on the code itself. Thank you!

@@ -35,4 +35,4 @@ Feature: Sizes
When I get the JSON
Then the JSON at "one" should have 1 entry
And the JSON at "two" should have 2 values
And the JSON at "three" should have 3 numbers
And the JSON at "three" should have 3 numbers
Copy link
Contributor

Choose a reason for hiding this comment

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

Please leave the newlines at the ends of files. Thank you!

@eluns
Copy link
Contributor Author

eluns commented Oct 6, 2014

Hi Steve,
Thank you for the feedback. I have made the corrections. Do you think I should add more tests?
Thanks again,
Beth

@laserlemon
Copy link
Contributor

Looks great, Beth. Would you please squash the commits where necessary. Because of the course correction, there are several commits which no longer apply and some commits are just tiny tweaks. Squashing the commits into one or two more feature-focused commits will help keep the Git history readable. Thank you!

@eluns
Copy link
Contributor Author

eluns commented Oct 6, 2014

Hi Steve,
I was able to squash the commits into 2 commits. Anything else I could do?
Thanks again for your time and help,
Beth

@laserlemon
Copy link
Contributor

No, looks great. Thank you. I'll merge shortly when I have time to ship a new release.

laserlemon added a commit that referenced this pull request Oct 10, 2014
Raise error when checking a size of a json ruby value of nil
@laserlemon laserlemon merged commit 794b747 into collectiveidea:master Oct 10, 2014
@laserlemon
Copy link
Contributor

Version 1.1.4 released. Thank you!

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