Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(docker): skip zoneinfo backport on newer python versions #5912

Merged
merged 1 commit into from
Sep 13, 2022
Merged

fix(docker): skip zoneinfo backport on newer python versions #5912

merged 1 commit into from
Sep 13, 2022

Conversation

hsheth2
Copy link
Collaborator

@hsheth2 hsheth2 commented Sep 12, 2022

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@github-actions github-actions bot added the devops PR or Issue related to DataHub backend & deployment label Sep 12, 2022
@github-actions
Copy link

Unit Test Results (build & test)

571 tests  ±0   571 ✔️ ±0   15m 20s ⏱️ +48s
141 suites ±0       0 💤 ±0 
141 files   ±0       0 ❌ ±0 

Results for commit 1df7944. ± Comparison against base commit 46bc1fe.

anshbansal
anshbansal approved these changes Sep 13, 2022
View reviewed changes
Copy link
Collaborator

@anshbansal anshbansal left a comment

Choose a reason for hiding this comment

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

LGTM

@hsheth2 hsheth2 merged commit aa93a34 into datahub-project:master Sep 13, 2022
@hsheth2 hsheth2 deleted the zoneinfo-backport branch September 13, 2022 18:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@anshbansal anshbansal anshbansal approved these changes

Assignees
No one assigned
Labels
devops PR or Issue related to DataHub backend & deployment
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hsheth2 @anshbansal