zrhoffman commented on a change in pull request #4630: Updating the testing
documentation for TO to add more info about buil…
URL: https://github.com/apache/trafficcontrol/pull/4630#discussion_r408370253
##########
File path: traffic_ops/testing/api/README.md
##########
@@ -41,7 +41,31 @@ In order to run the tests you will need the following:
and here: `traffic-ops-test.conf`
The Traffic Ops users will be created by the tool for accessing the API
once the database is accessible.
-
+
+ Note that for the database to successfully set up the tables and run the
migrations, you will need `goose`.
+ On your local machines(Macs), DO NOT install `goose` using `homebrew`. It
will install another non compliant
Review comment:
I wouldn't call it non-compliant, it's just the more popular
[pressly/goose](https://github.com/pressly/goose) fork instead of the
unmaintained https://bitbucket.org/liamstask/goose version that we use. Can
this change to saying that Homebrew installs an unsupported goose version, or
that Homebrew installs a different "goose"?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
