-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
fix: Updated city-timezones to support Kyiv #17159
Conversation
@AlexHladin is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details:
|
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (10/18/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add foundation team as reviewer" took an action on this PR • (10/18/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add community label" took an action on this PR • (10/18/24)1 label was added to this PR based on Keith Williams's automation. |
E2E results are ready! |
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.
Unit tests are failing
Head branch was pushed to by a user without write access
# Conflicts: # yarn.lock
541473e
to
12ec88f
Compare
Closing this PR as it is doing way more that describes. Feel free to re-open with only the relevant parts. Also feel free to open another one with type improvements if necessary. |
@AlexHladin Hey Alex, there's lots of type improvements going on in this PR. While we massively appreciate it, it makes it very difficult to review. Please open a fresh PR and request me a review. |
What does this PR do?
This PR fixes the incorrect timezone name for Kyiv.
Mandatory Tasks (DO NOT REMOVE)