Add verify_constant_names config option. #19

Merged
merged 2 commits into from Jul 10, 2012

Conversation

Projects
None yet
2 participants
Collaborator

myronmarston commented Jul 9, 2012

When this is set to true, an error will be raised if a given constant name is undefined.

This is the idea I had in #16.

If you like this, it'd be great to get it in the next release. If you have any concerns, feel free to hold off for now--I'd rather get a rspec-2.11-compatible release out sooner (I have a few projects I won't be able to upgrade rspec on until there's a corresponding rspec-fire release).

@myronmarston myronmarston Add verify_constant_names config option.
When this is set to true, an error will be raised if a given constant name is undefined.
4691ff7
Owner

xaviershay commented Jul 10, 2012

Well played adding new code just after you delete a heap ;)
Add something to the README, merge, then I'll push out a new release.

@myronmarston myronmarston added a commit that referenced this pull request Jul 10, 2012

@myronmarston myronmarston Merge pull request #19 from xaviershay/verify_constant_names
Add verify_constant_names config option.
77eb8dd

@myronmarston myronmarston merged commit 77eb8dd into master Jul 10, 2012

Collaborator

myronmarston commented Jul 10, 2012

Done :). Thanks!

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