Skip to content

Commit

Permalink
Merge pull request #1109 from lihenggui/fix/instructions
Browse files Browse the repository at this point in the history
Improve documentation and instructions
  • Loading branch information
lihenggui authored Dec 19, 2024
2 parents 5d33a1a + 58d47f5 commit d287e8e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/Build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,9 @@ jobs:
continue-on-error: false
if: steps.screenshotsverify.outcome == 'failure' && github.event.pull_request.head.repo.full_name != github.repository
run: |
echo "::error::Screenshot tests failed, please create a PR in your fork first." && exit 1
echo "::error::Screenshot tests failed, please create a PR in your fork first."
echo "Your fork's CI will take screenshots for your fork."
exit 1
# Runs if previous job failed
- name: Generate new screenshots if verification failed and it's a PR
Expand Down
3 changes: 2 additions & 1 deletion generateModuleGraphs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ then
echo "The 'dot' command is not found. This is required to generate SVGs from the Graphviz files."
echo "Installation instructions:"
echo " - On macOS: You can install Graphviz using Homebrew with the command: 'brew install graphviz'"
echo " - On Ubuntu: You can install Graphviz using APT with the command: 'sudo apt-get install graphviz'"
echo " - On Ubuntu: You can install Graphviz using APT with the command: 'sudo apt install graphviz'"
echo " - Others: Visit https://graphviz.org/download/"
exit 1
fi

Expand Down

0 comments on commit d287e8e

Please sign in to comment.