Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added noop sync test #19

Merged
merged 1 commit into from Sep 17, 2012
Merged

added noop sync test #19

merged 1 commit into from Sep 17, 2012

Conversation

phoolish
Copy link
Contributor

Sorry, had to create a new pull request because of the change in branch.

@benjaminoakes
Copy link
Member

Thanks for this.

I hadn't thought of the branch change making a difference (I probably could have still pulled from your master). Anyway, sorry about that.

This will most likely be going out in an upcoming release, along with some other low difficulty issues (any contributions are appreciated).

Also, if you'd like to help test the beta version of the gem that has sync, please drop a comment in #10.

@benjaminoakes benjaminoakes mentioned this pull request Sep 11, 2012
@benjaminoakes
Copy link
Member

Thanks! I'm merging this into simple-tools shortly...

benjaminoakes added a commit that referenced this pull request Sep 17, 2012
@benjaminoakes benjaminoakes merged commit ab28c9d into maid:sync Sep 17, 2012
@benjaminoakes
Copy link
Member

I could do a similar refactoring with Array() here too... it went from 9 to 3 lines. 😄

3780eef

Merging into simple-tools.

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

Successfully merging this pull request may close these issues.

None yet

2 participants