Skip to content

Conversation

@martonvago
Copy link
Contributor

@martonvago martonvago commented Nov 21, 2025

Description

This PR integrates cdp πŸŽ‰ .
Says many files, but most were just deleted πŸ”₯ .

Closes #1582

Needs an in-depth review.

Checklist

  • Formatted Markdown
  • Ran just run-all

@martonvago martonvago moved this from Todo to In Review in Iteration planning Nov 21, 2025
@martonvago martonvago marked this pull request as ready for review November 21, 2025 12:05
@martonvago martonvago requested a review from a team as a code owner November 21, 2025 12:05
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Some small comments, otherwise, looking goooodddd

@martonvago martonvago requested a review from lwjohnst86 December 5, 2025 11:09
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Looking gooodddd 😁 small comments about the error/replacing issue.

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Dec 8, 2025
@martonvago martonvago moved this from In Progress to In Review in Iteration planning Dec 8, 2025
@martonvago martonvago requested a review from lwjohnst86 December 8, 2025 12:12
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Amazing πŸŽ‰

@github-project-automation github-project-automation bot moved this from In Review to Approved in Iteration planning Dec 8, 2025
@lwjohnst86 lwjohnst86 merged commit b5b282b into main Dec 8, 2025
6 checks passed
@lwjohnst86 lwjohnst86 deleted the feat/integrate-cdp branch December 8, 2025 12:50
@github-project-automation github-project-automation bot moved this from Approved to Done in Iteration planning Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Integrate check-datapackage into Sprout

3 participants