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

Refactored Reachability with a three-letter prefix #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrRooni
Copy link

@MrRooni MrRooni commented May 20, 2013

I refactored the Reachability class to have a three-letter prefix as per Apple's recommendations. http://developer.apple.com/library/ios/#documentation/cocoa/conceptual/ProgrammingWithObjectiveC/Conventions/Conventions.html

Great work on this class, thanks for providing it!

best,
Michael Fey
michael@nothirst.com
No Thirst Software LLC

@tonymillion
Copy link
Owner

This would break its "drop in" replacement for the older Apple stuff. I'll leave this open though so others could use your patch.

@andrewtj
Copy link

andrewtj commented Jul 2, 2013

I don't have an interest in seeing this added myself but I think I've seen this requested before so maybe 318386e would be a useful addition.

@Coeur
Copy link

Coeur commented Feb 16, 2016

Hello, if your readme states that apps will get rejected with the current name, then drop in feature has to be dropped. I suggest TMReachability.

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

4 participants