Skip to content

chore: restructure packages of the definition module - #2813

Draft
rsynek wants to merge 9 commits into
TimefoldAI:mainfrom
rsynek:chore/definition-preview-api
Draft

rsynek wants to merge 9 commits into
TimefoldAI:mainfrom
rsynek:chore/definition-preview-api

Conversation

@rsynek

@rsynek rsynek commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

@winklerm Introducing the new structure; the build fails due to Revapi, which is expected at this point.

I will make further changes only if the new structure is reviewed and accepted.

@rsynek
rsynek requested a review from winklerm November 9, 2026 16:34

@winklerm winklerm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you, the structure looks very good to me!

Please see a few questions/comments inline.

@winklerm winklerm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you, looks good to me!

This branch was successfully deployed

1 active and 1 inactive deployments
documentation (preview) — b1a4266b Deployed Nov 12, 2026 by rsynek via Build Documentation #895
internal — b1a4266b Deployed Nov 12, 2026 by rsynek via approval_required #895
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.

2 participants