Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

Update installation instructions #184

Merged

Conversation

rhoggSugarcrm
Copy link
Contributor

Make it clearer that Thorn is meant to be used with Mocha.

@rhoggSugarcrm rhoggSugarcrm self-assigned this Jul 24, 2017

You will need to invoke Mocha in order to run Thorn. Check out
[Mocha's usage information][mocha-usage], [grunt-mocha], and
[gulp-spawn-mocha] for information about instantiating Mocha.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not providing Sugar-specific information here, just a general comment about how to invoke Thorn tests.

README.md Outdated
@@ -2,12 +2,6 @@

Thorn is composed of a set of helper abstractions built to ease the process of setting up a [SugarCRM](https://www.sugarcrm.com) instance's REST API testing environment and interacting with it.

## Installation
Copy link
Contributor

Choose a reason for hiding this comment

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

@rhoggSugarcrm why not keeping at least a reference to installation steps here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

TBH, it makes sense to put it back. I think I was considering being more explicit about how Sugar uses Thorn internally when I originally took this out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Restored!

@rhoggSugarcrm rhoggSugarcrm force-pushed the update-installation-instructions branch 2 times, most recently from 8d80b7f to 6ac70bd Compare August 31, 2017 17:17
@sugarcrm sugarcrm deleted a comment from coveralls Aug 31, 2017
@sugarcrm sugarcrm deleted a comment from coveralls Aug 31, 2017
@sugarcrm sugarcrm deleted a comment from coveralls Sep 5, 2017
@sugarcrm sugarcrm deleted a comment from coveralls Sep 19, 2017
Signed-off-by: Bob Wombat Hogg <rhogg@sugarcrm.com>
Signed-off-by: Bob Wombat Hogg <rhogg@sugarcrm.com>
@coveralls
Copy link

coveralls commented Sep 27, 2017

Coverage Status

Coverage remained the same at 95.549% when pulling 2aad34b on rhoggSugarcrm:update-installation-instructions into ecbcd35 on sugarcrm:master.

Copy link
Contributor

@jcsmorais jcsmorais left a comment

Choose a reason for hiding this comment

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

:shipit:

@jcsmorais jcsmorais merged commit 55a5a84 into sugarcrm:master Sep 27, 2017
@rhoggSugarcrm rhoggSugarcrm deleted the update-installation-instructions branch September 27, 2017 20:41
@sugarcrm sugarcrm deleted a comment from coveralls Sep 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants