Skip to content

Conversation

@mralexgray
Copy link
Contributor

Implemented most opportunities to use Blocs-based API's. everything works AOK. initial work to port to Cocoa done, too.

140329-0001

Implemented most opportunities to use Blocs-based API's. everything
works AOK.  initial work to port to Cocoa done, too.
@pwightman
Copy link
Owner

Hey Thanks! Glad to see someone giving this some love. What do you think about me transferring ownership of this repository to you? I built this as a proof-of-concept, but don't actively use it for anything so I'd just as soon give it to someone that might be making use of it. Your coding style is also very different from mine, so by introducing this change, I don't think I'd feel much ownership in it anymore anyway (which I'm totally fine with). So my proposal is merging this, then transferring ownership. Thoughts?

Copy link
Owner

Choose a reason for hiding this comment

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

Missing an 's' here on section?

@mralexgray
Copy link
Contributor Author

Sure sounds ok! I've almost got it working on Mac OS X. Would be nice to have a solution for something like this that was API compatible on both platforms.

pwightman pushed a commit that referenced this pull request Mar 30, 2014
ARC + modernized + refactored + simplified
@pwightman pwightman merged commit 3f2b429 into pwightman:master Mar 30, 2014
@pwightman
Copy link
Owner

Great! I've merged, but I can't transfer the repo cause of your fork. Delete your fork and then I'll transfer it over.

Thanks!

@mralexgray mralexgray deleted the modern branch March 30, 2014 22:44
@mralexgray
Copy link
Contributor Author

Deleted branch, AOK!

@pwightman
Copy link
Owner

It has to be your entire repository, not just the branch. It won't let me transfer because you already have a repository called PDTiledView (which is just your fork of my repository). You can delete it in settings, pictured here:

image

@mralexgray
Copy link
Contributor Author

Ok, I have a feeling this whole thread might disappear once I do that.. maybe, but here goes!

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.

2 participants