Feat: Added Notifications section to notify on failed python test bui…#78
Open
Hemantrathore20 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
Open
Feat: Added Notifications section to notify on failed python test bui…#78Hemantrathore20 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
Hemantrathore20 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a group submission by Hemant Rathore, Manal Ghanem and Boris Gunn.
[X] New feature
Changes
Added a new notification section in testPython.yaml file that sends out an email notifications to addressees when there are failures with the test builds.
Testing
After changes are pushed to my main branch, triggered python test workflow job. It sends out a failure email to addresses configured, also in case of no failures, it does not send out any notifications.
Screenshots
Dependencies
This change requires three environment variables to be setup in github "Secrets" section. The three variables are as below
secrets.MAIL_USERNAME = "Your email account user name (possibly gmail)"
secrets.MAIL_PASSWORD = "Your email account password"
secrets.NOTIFY_EMAIL = "List of addressees to notify in comma separated list."
Breaking Changes
None