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

Fix broken links in README #28

Merged
merged 1 commit into from Mar 25, 2015
Merged

Fix broken links in README #28

merged 1 commit into from Mar 25, 2015

Conversation

jgorset
Copy link
Contributor

@jgorset jgorset commented Mar 25, 2015

Yes.

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

#27 :(

zenangst added a commit that referenced this pull request Mar 25, 2015
@zenangst zenangst merged commit 0e0f25e into master Mar 25, 2015
@zenangst zenangst deleted the fix/links branch March 25, 2015 20:39
@jgorset
Copy link
Contributor Author

jgorset commented Mar 25, 2015

Haha, oh no!

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

@zenangst ....

@jgorset
Copy link
Contributor Author

jgorset commented Mar 25, 2015

I WIN

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

👍

@zenangst
Copy link
Contributor

@jgorset pointed to master

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

Pointing to master is not better

@zenangst
Copy link
Contributor

It's not?

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

As soon as you change one line of code the URL is no longer valid.

I this case moving the folder

@zenangst
Copy link
Contributor

Oh... well... that's awkward.

@jgorset
Copy link
Contributor Author

jgorset commented Mar 25, 2015

That… that's true.

@zenangst
Copy link
Contributor

ron-swanson-computer-throw-out-parks-and-rec

@3lvis
Copy link
Owner

3lvis commented Mar 25, 2015

Might be even better to point to a tag/release! In case you break API

https://github.com/hyperoslo/Sync/tree/0.6/Examples/AppNet

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

3 participants