-
Notifications
You must be signed in to change notification settings - Fork 442
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.
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
Email notification #80
Conversation
…ings from the phpci config under the heading email_settings: smtp_address smtp_port smtp_username smtp_password and from_address
… out if the the build has succeeded or not.
… these addresses will be mailed with a pass or fail message.
swiftmailer was added automatically by PHPStorm and the indentation didn't match the rest of the file.
Looks good to me! What makes it a partial implementation? |
The three main things that need adding are:
|
Ah, good call. Are we going to leave un-merged until that stuff is done? |
Hmmm. Hadn't really thought about it. It works (or at least should) in very simple cases so it might be worth merging now to start getting some testing of it done. I don't anticipate any major structural changes so merging now and then merging again later shouldn't affect very much. |
Partially implements functionality for #35.
Email notifications sent to a default address registered during install and any addresses added in the phpci.yml:
complete:
email:
addresses:
- test1@gmail.com
- test2@gmail.com