Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Update changelog and cleanup comments #938

Closed
wants to merge 2 commits into from

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Jun 10, 2013

Got time for a quick overview @myronmarston ?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling fe890e1 on JonRowe:memoized_cleanup into 3efa3fd on rspec:master.


Enhancements

* Clean up some internal use of Enumerable methods. (Vipul A M)

Bug fixes

* Refactor memoized helpers (let, subject, named subject) to use an internalised
Copy link
Member

Choose a reason for hiding this comment

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

As an American, I'd usually spell this "internalized" but given that we have you and @samphippen on the team now this is fine :).

@myronmarston
Copy link
Member

Hold off on merging this. I have a better fix coming in a PR in a few minutes :).

@myronmarston
Copy link
Member

Closing in preference for #942.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants