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

Updated code owners and disabled dependency to ucx #52

Merged
merged 1 commit into from
Aug 19, 2024
Merged

Updated code owners and disabled dependency to ucx #52

merged 1 commit into from
Aug 19, 2024

Conversation

mwojtyczka
Copy link
Contributor

@mwojtyczka mwojtyczka commented Aug 8, 2024

Changes

  • updated code owners
  • disabled dependency to ucx

Linked issues

Resolves #..

Tests

  • manually tested
  • added unit tests
  • added integration tests

@mwojtyczka mwojtyczka requested a review from nfx as a code owner August 8, 2024 13:21
@mwojtyczka mwojtyczka requested a review from alexott August 8, 2024 13:22
@mwojtyczka mwojtyczka changed the title disabled ucx dependency Updated code owners Aug 8, 2024
@mwojtyczka mwojtyczka changed the title Updated code owners Updated code owners and disabled dependency to ucx Aug 8, 2024
@alexott
Copy link
Contributor

alexott commented Aug 15, 2024

@nfx can you approve it?

@alexott alexott enabled auto-merge (squash) August 15, 2024 18:36
@nehamilak-db nehamilak-db self-requested a review August 19, 2024 08:45
Copy link
Contributor

@nehamilak-db nehamilak-db left a comment

Choose a reason for hiding this comment

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

Looks good.

Copy link

@gergo-databricks gergo-databricks left a comment

Choose a reason for hiding this comment

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

LGTM+1

@mwojtyczka mwojtyczka force-pushed the deps branch 2 times, most recently from b4cc6ea to 0605b5c Compare August 19, 2024 13:57
@alexott alexott merged commit de749a5 into main Aug 19, 2024
6 checks passed
@alexott alexott deleted the deps branch August 19, 2024 14:04
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.

4 participants