-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add Missing Timezone to RS #17037
base: main
Are you sure you want to change the base?
Add Missing Timezone to RS #17037
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
@@ -62,15 +62,22 @@ enum USTimeZone { | |||
ARIZONA = "ARIZONA", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need more research for Zone Name.
Test Results1 272 tests +7 1 268 ✅ +7 7m 30s ⏱️ -8s Results for commit 9a3fc79. ± Comparison against base commit c942a9a. This pull request removes 1 and adds 8 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Integration Test Results 60 files 60 suites 44m 15s ⏱️ Results for commit 9a3fc79. ♻️ This comment has been updated with latest results. |
Branch deployed to Chromatic 🚀.
View via: |
|
Quality Gate passedIssues Measures |
This PR adds missing timezone to RS.
Test Steps:
Changes
Checklist
Testing
./prime test
or./gradlew testSmoke
against local Docker ReportStream container?npm run lint:write
?Process
Linked Issues
To Be Done
Create GitHub issues to track the work remaining, if any
Specific Security-related subjects a reviewer should pay specific attention to
If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least: