Skip to content

Commit

Permalink
Pip check instructs to ask a TA for help
Browse files Browse the repository at this point in the history
Previously we instructed to `pip install` the
missing packages. This leads to a very different
setup than we aim for, because no version numbers
are provided.

Changed this to:
- list the packages
- instruct the students to call in a TA
- note to TA to redo the Python packages step
  (usually the root cause is that Apple users
  skipped that step)
  • Loading branch information
julesvanrie committed Oct 26, 2024
1 parent a04c052 commit 611021d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions checks/pip_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,11 @@ for r in ${REQUIRED[@]}; do
fi
done
if (( ${#missing[@]} )); then
sentence='Try to `pip install '
sentence='❌ Some packages are missing: '
sentence+=$missing
sentence+='` again.'
echo '❌ Some packages are missing:'
echo $sentence
echo '👉 Ask a TA for help to check your python setup.'
echo ' Note to TAs: First thing to do: redo the Python packages step of the instructions.'
exit 1
else
echo '✅ Everything is fine, continue the setup instructions.'
Expand Down

0 comments on commit 611021d

Please sign in to comment.