Skip to content

Conversation

glebuk
Copy link
Contributor

@glebuk glebuk commented May 8, 2018

This fixes an issue introduced by PR#52
Updates the path to a sentiment sample file in README.MD

@justinormont
Copy link
Contributor

justinormont commented May 8, 2018

Should our URLs in README.md, et al. be relative links? Currently, they won't track forks & branches. #Resolved

@glebuk
Copy link
Contributor Author

glebuk commented May 8, 2018

fixed


In reply to: 387327793 [](ancestors = 387327793)

@glebuk glebuk requested a review from TomFinley May 8, 2018 15:06
Copy link
Contributor

@TomFinley TomFinley left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Member

@eerhardt eerhardt left a comment

Choose a reason for hiding this comment

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

LGTM

@TomFinley TomFinley merged commit e9942bb into dotnet:master May 8, 2018
eerhardt pushed a commit to eerhardt/machinelearning that referenced this pull request Jul 27, 2018
* Fix link to example file in readme.md

* replaced URLs with relative paths.

* fixed link
@ghost ghost locked as resolved and limited conversation to collaborators Mar 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants