Move core extensions to lib/core_ext.rb #44

Merged
merged 3 commits into from Mar 10, 2014

Projects

None yet

2 participants

@blambeau
Collaborator
blambeau commented Mar 6, 2014

According to the twitter discussion. As such this breaks Citrus public API, since the file is not required by default.

@mjackson
Owner

@blambeau Let's go ahead and merge this, though I think that lib/citrus/core_ext.rb should require 'citrus``at the top. This lets us remove therequire 'citrus'intest/helper.rb` and simplifies usage for end users as well.

@blambeau
Collaborator

Done. Btw, this is a (not so serious) broken API. Are you planning to release 3.0 then?

@mjackson
Owner

@blambeau Yes, but let's wait to release 3.0 when we get #41 sorted out as well since it will also break the public API.

@mjackson mjackson merged commit fc9ec17 into mjackson:master Mar 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment