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

Replay Alex Chaffee's commits on top of current master #34

Closed
wants to merge 19 commits into from
Closed

Replay Alex Chaffee's commits on top of current master #34

wants to merge 19 commits into from

Conversation

cantino
Copy link

@cantino cantino commented Aug 16, 2013

This PR replays Alex's work from 2 years ago on adding QuestionForm support to rturk, as well as some of his other small refactors. I left out the rspec2 upgrade because that seemed to be receiving pushback. Specs are all passing.

Conflicts:
	spec/operations/create_hit_spec.rb
…ency with its real name (MaxAssignments) and to remove confusion between it and the other property named 'assignments'
…ke either a QF or just its content; fix another method_missing problem

Conflicts:
	lib/rturk/parsers/responses/get_hit_response.rb
Conflicts:
	lib/rturk/builders/qualification_builder.rb
Conflicts:
	lib/rturk/builders/qualification_builder.rb
@mdp
Copy link
Collaborator

mdp commented Aug 16, 2013

This looks good, however it's exposed a problem with Nokogiri not working on 1.8.7. So from this release forward I'm going to just support 1.9.2 and up.

I'll release 2.10.3 as the last ruby 1.8.7 build, and I'll bump the release to 2.11.0 to include this PR and only be Ruby >1.9.2 compatible.

I'll have this in first thing tomorrow morning when I get more time.

Thanks for taking the time to get this in.

@cantino
Copy link
Author

cantino commented Aug 17, 2013

Sounds great @mdp! This PR does contain a few changes, such as renamed fields and the renamed Question to ExternalQuestion, but I think it's a good improvement from @alexch.

@mdp mdp closed this in 0bcabf2 Aug 19, 2013
@mdp
Copy link
Collaborator

mdp commented Aug 19, 2013

Out in 2.11.0, thanks @cantino and @alexch

@cantino
Copy link
Author

cantino commented Aug 19, 2013

Thanks @mdp!

@alexch
Copy link
Collaborator

alexch commented Aug 19, 2013

Yay!

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.

4 participants