Skip to content

Commit

Permalink
Remove docker command from make cli.
Browse files Browse the repository at this point in the history
  • Loading branch information
jimmygchen committed Oct 21, 2024
1 parent b88cb8c commit 7baaacd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 9 deletions.
8 changes: 1 addition & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -183,15 +183,9 @@ test-exec-engine:
# test vectors.
test: test-release

# Updates the CLI help text pages in the Lighthouse book, building with Docker.
cli:
docker run --rm --user=root \
-v ${PWD}:/home/runner/actions-runner/lighthouse sigmaprime/github-runner \
bash -c 'cd lighthouse && make && ./scripts/cli.sh'

# Updates the CLI help text pages in the Lighthouse book, building using local
# `cargo`.
cli-local:
cli:
make && ./scripts/cli.sh

# Check for markdown files
Expand Down
4 changes: 2 additions & 2 deletions scripts/cli.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

# IMPORTANT
# This script should NOT be run directly.
# Run `make cli` or `make cli-local` from the root of the repository instead.
# Run `make cli` from the root of the repository instead.

set -e

Expand Down Expand Up @@ -90,7 +90,7 @@ rm -f help_general.md help_bn.md help_vc.md help_am.md help_vm.md help_vm_create

# only exit at the very end
if [[ $changes == true ]]; then
echo "Exiting with error to indicate changes occurred. To fix, run 'make cli-local' or 'make cli' and commit the changes."
echo "Exiting with error to indicate changes occurred. To fix, run 'make cli' and commit the changes."
exit 1
else
echo "CLI help texts are up to date."
Expand Down

0 comments on commit 7baaacd

Please sign in to comment.