Skip to content
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

Update PERL email script name and parameter list. #26244

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tzinsky
Copy link
Collaborator

@tzinsky tzinsky commented Nov 15, 2024

This change helps to explains what function this code now performs. The code does not send emails but instead creates an email file that is picked up by Jenkins to send to the team when a test fails. This change is to be merged before the module test code.

This change also renames the script to reflect what it is actually
doing. The code does not send emails but instead creates an email
file that is picked up by Jenkins to send to the team when a test
fails.

Signed-off-by: Tim Zinsky <[email protected]>
util/cron/nightly_email_file.pm Outdated Show resolved Hide resolved
@riftEmber
Copy link
Member

LGTM, but please hold off on merging until we get the chaos from LLVM 19 upgrade quieted down

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants