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

Merges #41 #42

Merged
merged 5 commits into from
Aug 31, 2017
Merged

Merges #41 #42

merged 5 commits into from
Aug 31, 2017

Conversation

ashfurrow
Copy link
Member

@ashfurrow ashfurrow commented Aug 31, 2017

Merges #41, fixes the CI issue. Please see the individual commits cb1368f, fd85abf, and b8a1d88
for specifics. In summary:

  • Moves tests added in disposeBag: add a HasDisposeBag protocol #41 to Demo unit test folder.
  • Moves source files back to the root of the repo.
  • Undid some changes to the Carthage Xcode project.
  • Added a note in the readme to ward off this issue in the future.

Thibault Wittemberg and others added 5 commits August 31, 2017 09:07
In this commit, we try to conform to protocol oriented programming. It is more flexible to
introduce a HasDisposeBag protocol so that every object that respect this one
can have a DisposeBag (NSObject does respect that protocol now)
@ashfurrow ashfurrow merged commit 2ffa937 into master Aug 31, 2017
@ashfurrow ashfurrow deleted the twittemb-feature/disposeBagable branch August 31, 2017 16:28
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.

1 participant