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

Add setup instructions for Carthage #7

Closed
wants to merge 1 commit into from

Conversation

Palleas
Copy link

@Palleas Palleas commented Apr 14, 2015

@robb
Copy link

robb commented Apr 14, 2015

FWIW, it's already compatible, so adding

github "ResearchKit/ResearchKit" "stable"

to your Cartfile is enough to get started.

@Palleas
Copy link
Author

Palleas commented Apr 14, 2015

I figured targeting "stable" would be better since there is no tags (yet?).

@robb
Copy link

robb commented Apr 14, 2015

Whoops, totally missed that this is a Pull Requests with a change to the README 😅


````
github "ResearchKit/ResearchKit" "stable"
````
Copy link

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 😄

Copy link
Author

Choose a reason for hiding this comment

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

Fixed, thx!

@ghost
Copy link

ghost commented Apr 15, 2015

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.

@jwe-apple
Copy link
Member

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.

@Palleas
Copy link
Author

Palleas commented Apr 15, 2015

Let me know if I can help with this page! 👍

@jwe-apple
Copy link
Member

This has already been done via PR #5

@jwe-apple jwe-apple closed this Apr 18, 2015
@Palleas Palleas deleted the patch-1 branch May 6, 2015 07:51
httang12 pushed a commit to httang12/ResearchKit that referenced this pull request Jul 24, 2015
…/SOL-SOL-1476-Dismiss-old-notif to develop

* commit '54281968fb348766a8a6d55c420900ad5240f1f8':
  Notification texts changed
  Old notifications removed
Erin-Mounts added a commit to Erin-Mounts/ResearchKit that referenced this pull request Feb 26, 2016
Assorted nit fixes required for onboarding
umerkhan-apple pushed a commit that referenced this pull request Jan 25, 2017
Implement requested changes in Video Instruction Step
srinathtm-apple pushed a commit that referenced this pull request Feb 10, 2020
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