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
Adds deploy docs #15
Adds deploy docs #15
Conversation
Signed-off-by: Tallys Martins <tallysmartins@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comment looks great otherwise
@@ -1,4 +1,4 @@ | |||
<img src="../web/public/images/logo.png" height="68" /> | |||
<img src="https://github.com/elixir-bench/elixir-bench.github.io/blob/master/images/logo.png" height="68" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
README.md
Outdated
environment variables for communication with the API server. | ||
This project uses `distillery` for deployments. The relese requires a `RUNNER_API_URL`, | ||
`RUNNER_API_KEY` and `RUNNER_API_USER` environment variables for communication with | ||
the API server. Built releases are placed under `_build/prod/rel/runner` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this just duplicates the information from the table below, which seems much better imo :) (aka I'd remove the mention of the envs here)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 removed!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
? They're still up there? The place were you just removed them I actually liked :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh sorry! bad reading 😮 I've just put it back! 👍 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed now ;)
Signed-off-by: Tallys Martins <tallysmartins@gmail.com>
README.md
Outdated
|
||
```bash | ||
RUNNER_API_URL=api.elixirbench.org/runner-api RUNNER_API_USER=test-runner RUNNER_API_KEY=test _build/prod/rel/runner/bin/runner foreground |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually liked having them here
Signed-off-by: Tallys Martins <tallysmartins@gmail.com>
No description provided.