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

@orta => Replaced NATiledImageView with ARTiledImageView #26

Merged
merged 5 commits into from
Mar 17, 2014
Merged

@orta => Replaced NATiledImageView with ARTiledImageView #26

merged 5 commits into from
Mar 17, 2014

Conversation

dblock
Copy link
Collaborator

@dblock dblock commented Mar 17, 2014

Use the ARTiledImageView pod, as well as a number of misc fixes and improvements.

*/
@interface NADotAnnotation : NAAnnotation

// Dot radius.
Copy link
Collaborator

Choose a reason for hiding this comment

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

///

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like they all need this, after accepting this PR I'll do a documentation run, http://cocoadocs.org/docsets/NAMapKit/3.0/Classes/NAMapView.html

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thx!

@orta
Copy link
Collaborator

orta commented Mar 17, 2014

also travis doesn't seem to be running on this PR, is this a bug?

@dblock
Copy link
Collaborator Author

dblock commented Mar 17, 2014

looking at CI, not sure what's up

@dblock
Copy link
Collaborator Author

dblock commented Mar 17, 2014

@orta CI still runs XCode 5.0.2 and those screenshots aren't the same. There's travis-ci/travis-ci#2051 to update that and there's facebookarchive/ios-snapshot-test-case#19 that seems to address this as well. I think this is safe to merge, locally all these tests pass for me, we'll fix them before the next release one way or another, of course.

@orta
Copy link
Collaborator

orta commented Mar 17, 2014

Sure.

orta added a commit that referenced this pull request Mar 17, 2014
@orta => Replaced NATiledImageView with ARTiledImageView
@orta orta merged commit 8c475da into neilang:master Mar 17, 2014
@dblock dblock deleted the artiledimageview branch March 17, 2014 21:56
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