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

Allowing access to the OS version #254

Merged
merged 3 commits into from
Nov 6, 2013
Merged

Allowing access to the OS version #254

merged 3 commits into from
Nov 6, 2013

Conversation

seanoshea
Copy link
Contributor

Allowing access to the OS version

ondrejhanslik added a commit that referenced this pull request Nov 6, 2013
Allowing access to the OS version
@ondrejhanslik ondrejhanslik merged commit 34ee2e1 into moredip:master Nov 6, 2013
@seanoshea
Copy link
Contributor Author

Awesome. Thanks.

@moredip
Copy link
Owner

moredip commented Nov 6, 2013

Thanks @ondrejhanslik.

For future reference, I'd prefer us using the new obj-C literals for arrays and dicts. They are supported on all runtimes and reduce the amount of boilerplate quite a bit.

@seanoshea
Copy link
Contributor Author

Happy to submit a new PR with literals. I was just sticking to the current syntax and didn't want to rock the boat too much with new syntax. I'd be the first to admit that I'm not all that familiar with mac development either, so I played it safe and just stuck with NSDictionary/NSArray syntax.

@moredip
Copy link
Owner

moredip commented Nov 7, 2013

No worries. If you feel motivated I'd be very happy to take the PR.

@seanoshea
Copy link
Contributor Author

Hey Pete. See #255 for an updated PR.

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.

3 participants