-
Notifications
You must be signed in to change notification settings - Fork 88
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
Enhancement: Trigger project status notifications based on scancode-config.yml values #1208
Comments
@DennisClark I need your help providing the layout and text content of the email notifications. |
Signed-off-by: tdruez <[email protected]>
@DennisClark I still need your input on the above to make more progress. |
email subject: email text: |
@pombredanne please comment on this from a functionality perspective: do we even need this ? If the results of the command-line scan are written to a specified location anyway, perhaps the next thing to do is to open in ScanCode.io if you want more information. Perhaps an email with the first few elements, down to the log file (perhaps included in the body of the email) would be sufficient. |
This feature would only make sense for a ScanCode.io server setup as it requires an email service provider to be configured. We should promote the, already implemented, usage of Webhook over email as it does not require any extra service to be configured and available. We are lacking documentation though, see #1027 |
@DennisClark I agree that we may not need this just now. |
See related issue about the scancode-config.yml file: #1197
If a project is created and executed with a scancode-config.yml file, and if that file contains a notification_address, send a notification (email or similar) to that address containing:
The text was updated successfully, but these errors were encountered: