Skip to content

Conversation

@onetechnical
Copy link
Contributor

@onetechnical onetechnical commented Jun 25, 2025

Summary

The CI pipeline for PRs and nightly tests were revised. This PR makes the following changes:

  • Fix SHORTTEST flag for e2e_subs.
  • Re-configure nightlies to better manage concurrency
  • Make slack webhook optional
  • Add test result uploads
  • Limit build uploads to targeted branches
  • Add workflow dispatch for nightly jobs

Test Plan

Verified upload of test files for both PRs and nightly runs

@codecov
Copy link

codecov bot commented Jun 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.25%. Comparing base (310a900) to head (ef628b9).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6372      +/-   ##
==========================================
- Coverage   50.60%   50.25%   -0.35%     
==========================================
  Files         653      653              
  Lines      110395   111124     +729     
==========================================
- Hits        55864    55847      -17     
- Misses      51657    52431     +774     
+ Partials     2874     2846      -28     

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

Copy link
Contributor

@gmalouf gmalouf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few questions/comments, but looks reasonable.

@onetechnical onetechnical merged commit ecc67c3 into algorand:master Jun 26, 2025
41 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants