Skip to content

Conversation

@asikowitz
Copy link
Collaborator

Looked thru breaking changes, nothing seemed scary to me

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

@asikowitz asikowitz requested a review from hsheth2 December 2, 2024 20:16
@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Dec 2, 2024
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Dec 2, 2024
@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Dec 2, 2024
@asikowitz asikowitz merged commit ff385ed into datahub-project:master Dec 17, 2024
72 of 73 checks passed
@asikowitz asikowitz deleted the bump-sqlglot-pin branch December 17, 2024 11:49
sleeperdeep pushed a commit to sleeperdeep/datahub that referenced this pull request Dec 17, 2024
brock-acryl pushed a commit to brock-acryl/datahub that referenced this pull request Jan 30, 2025
kartikey-visa pushed a commit to kartikey-visa/datahub that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants