Skip to content

Add notification feature #62

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.

Sign up for GitHub

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

Open
vinw12 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add notification feature #62

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions .github/workflows/testsPython.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,16 @@ jobs:
notifications:
needs: python-unit-tests
runs-on: ubuntu-latest
if: always() # Add this line
steps:
- name: Notify on test results
run: |
if [ "${{ needs.python-unit-tests.result }}" == "success" ]; then
echo "success notifications go here"
echo "::notice title=Tests Passed::All Python tests passed successfully! ✅"
echo "## ✅ Test Success" >> $GITHUB_STEP_SUMMARY
echo "All tests passed successfully" >> $GITHUB_STEP_SUMMARY
else
echo "failure notifications go here"
echo "::error title=Tests Failed::Some Python tests failed. ❌"
echo "## ❌ Test Failure" >> $GITHUB_STEP_SUMMARY
echo "Some tests failed." >> $GITHUB_STEP_SUMMARY
Comment on lines +78 to +83
Comment on lines 76 to +83
fi
Loading