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

ci(Mergify): configuration update #7720

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Conversation

KenLSM
Copy link
Contributor

@KenLSM KenLSM commented Sep 25, 2024

This change has been made by @KenLSM from the Mergify workflow automation editor.

Signed-off-by: Ken Lee Shu Ming <[email protected]>
@KenLSM KenLSM requested a review from kevin9foong September 25, 2024 07:12
@datadog-opengovsg
Copy link

Datadog Report

Branch report: mergify/KenLSM/config-update
Commit report: 77be3dc
Test service: formsg

❌ 2 Failed (0 Known Flaky), 2996 Passed, 1 Skipped, 1m 33.03s Total Time

❌ Failed Tests (2)

  • Verification service sendNewOtp form field should forward errors returned by SmsFactory.sendVerificationOtp - src/app/modules/verification/__tests__/verification.service.spec.ts

  • Verification service sendNewOtp form field should return TransactionNotFoundError when database update returns null - src/app/modules/verification/__tests__/verification.service.spec.ts - Details

    Expand for error
     Cannot read properties of undefined (reading 'isErr')
    

@kevin9foong kevin9foong merged commit df2e6b8 into develop Sep 25, 2024
21 of 23 checks passed
@kevin9foong kevin9foong deleted the mergify/KenLSM/config-update branch September 25, 2024 07:46
@kevin9foong kevin9foong mentioned this pull request Sep 26, 2024
46 tasks
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