Skip to content

Conversation

richsalz
Copy link
Collaborator

No need to change the test, keep as-is for compatibility.

There are 34 calls to send_mail in ietf/doc/mails.py. I don't think we want to review and possibly edit all of them,
wait until people file issues?

Fixes: #8526

@jennifer-richards jennifer-richards changed the title Add IETF to "Last Call Expired" email Subject fix: Add IETF to "Last Call Expired" email Subject Mar 15, 2025
@jennifer-richards jennifer-richards changed the title fix: Add IETF to "Last Call Expired" email Subject feat: Add IETF to "Last Call Expired" email Subject Mar 15, 2025
@richsalz
Copy link
Collaborator Author

The test failure seems unrelated? If not, please give me a clue what to fix.

Copy link

codecov bot commented Mar 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.80%. Comparing base (70bf2ae) to head (8f0b730).
Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8675      +/-   ##
==========================================
- Coverage   88.88%   88.80%   -0.08%     
==========================================
  Files         312      314       +2     
  Lines       40891    41221     +330     
==========================================
+ Hits        36345    36608     +263     
- Misses       4546     4613      +67     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rjsparks rjsparks merged commit 603938a into ietf-tools:main Mar 15, 2025
10 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minor improvement to IETF last call emails
2 participants