Made public CalendarBase and Calendar methods chainable #325

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@ghost
Contributor

ghost commented Oct 25, 2012

Also fixed a little bit of whitespace

Made public CalendarBase and Calendar methods chainable
Also fixed a little bit of whitespace
Contributor

allenrabinovich commented Oct 25, 2012

This is quite nice, thank you for contributing this. @asystance, can you also add unit tests that check for proper chaining (e.g., calling a few chainable methods in a chain and checking that the resulting changes are correct?).

@ghost
Contributor

ghost commented Oct 26, 2012

Sure, I'll write tests and figure out how to run them, and see whether the new lines are covered.

@ghost ghost closed this Oct 26, 2012

@ghost
Contributor

ghost commented Oct 31, 2012

@allenrabinovich did you take a look at #326 ?

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