Hey I was just poking around and I saw some t's to cross and so on.
Most significant changes were reworking control flow such that we don't need Scope::NONE, and removing unused arguments to DSL::FieldGroup and Search::FieldGroup.
Nothing I did should change the internal function of the library, and the tests run green except for an error related to posts_helper in sunspot-rails 2.3.14.
Remove redundant flatten - use comment rather than local var
Remove unnecessary use of NONE
Remove last remaining use of Scope::NONE
Move assignment inside the block it's used
Do the field setup once
Setup the field a bit earlier
Remove extraneous begin/end
Sunspot::Search::FieldGroup is never called with options, so remove t…
…he variable throughout
Sunspot::DSL::FieldGroup never makes use of @query - drop it throughout