Skip to content

Conversation

@jloh02
Copy link
Member

@jloh02 jloh02 commented Oct 8, 2025

To merge on 13 Oct 2025 12am

@vercel
Copy link

vercel bot commented Oct 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
nusmods-export Ready Ready Preview Comment Oct 12, 2025 4:01pm
nusmods-website Ready Ready Preview Comment Oct 12, 2025 4:01pm

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 26.66667% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.11%. Comparing base (988c6fd) to head (7bbb83a).
⚠️ Report is 145 commits behind head on master.

Files with missing lines Patch % Lines
website/src/views/mpe/MpeContainer.tsx 0.00% 8 Missing ⚠️
website/src/serverless/handler.ts 0.00% 1 Missing ⚠️
website/src/serverless/nus-auth.ts 0.00% 1 Missing ⚠️
website/src/views/mpe/constants.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4242      +/-   ##
==========================================
- Coverage   54.52%   53.11%   -1.42%     
==========================================
  Files         274      289      +15     
  Lines        6076     6693     +617     
  Branches     1455     1639     +184     
==========================================
+ Hits         3313     3555     +242     
- Misses       2763     3138     +375     

☔ 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.

Copy link
Member

@ravern ravern left a comment

Choose a reason for hiding this comment

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

LGTM pls merge at the right time

@leslieyip02 leslieyip02 enabled auto-merge (squash) October 12, 2025 16:00
@leslieyip02 leslieyip02 merged commit 00d7b81 into master Oct 12, 2025
4 of 6 checks passed
@leslieyip02 leslieyip02 deleted the cpex-staging branch October 12, 2025 16:01
@jloh02 jloh02 restored the cpex-staging branch October 17, 2025 12:28
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.

3 participants