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

install guide updates #3311

Closed
wants to merge 1 commit into from

Conversation

AmandaBirmingham
Copy link
Contributor

Some very small changes to clarify bits of the install guide, based on a trial usage :)

@@ -181,7 +181,7 @@ github organization. Each of the plugins have their own installation instruction
suggest looking at each individual .travis.yml file to see detailed installation
instructions. Note that the most common plugins are:
- [qtp-biom](https://github.com/qiita-spots/qtp-biom)
- [qtp-target-gene](https://github.com/qiita-spots/qtp-target-gene)
- [qtp-sequencing](https://github.com/qiita-spots/qtp-sequencing)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

qtp-target-gene has been renamed to qtp-sequencing. The old link still works, but it is surprising (and potentially worrying :) to the reader to end up at a different URL than the one they expected.

@@ -206,9 +206,7 @@ Set your `QIITA_CONFIG_FP` environment variable to point to that file (into `.ba
source activate qiita
```

Update paths in the newly copied configuration file to match your settings, e.g. replace /home/travis/ with your user home directory.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are no travis paths in the config anymore.

Update paths in the newly copied configuration file to match your settings, e.g. replace /home/travis/ with your user home directory.

If you are working on WSL, you will need to start the redis server with the following command before making a test environment:
If you are working on Mac OS X or WSL, you will need to start the redis server with the following command before making a test environment:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Install on mac (not just WSL) fails if one doesn't run the redis-server start ... ask me how I know :)

@AmandaBirmingham AmandaBirmingham changed the base branch from master to dev August 11, 2023 21:27
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.

1 participant