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
Add setup instructions for Carthage #7
Conversation
FWIW, it's already compatible, so adding
to your Cartfile is enough to get started. |
I figured targeting "stable" would be better since there is no tags (yet?). |
Whoops, totally missed that this is a Pull Requests with a change to the README 😅 |
|
||
```` | ||
github "ResearchKit/ResearchKit" "stable" | ||
```` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Picky, but code fences are delimited by three \
’s, not four 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thx!
Not sure what Apple’s policy is (will be?) regarding badges in README.md, but projects can also use this badge to declare they’re compatible with Carthage. |
Thanks for this suggestion. We're thinking of putting dependency management on a separate page, linked from the readme, rather than cluttering up the main page with it. Holding this open to track. |
Let me know if I can help with this page! 👍 |
This has already been done via PR #5 |
…/SOL-SOL-1476-Dismiss-old-notif to develop * commit '54281968fb348766a8a6d55c420900ad5240f1f8': Notification texts changed Old notifications removed
Assorted nit fixes required for onboarding
Implement requested changes in Video Instruction Step
Cf. https://github.com/Carthage/Carthage