Skip to content

refactor/move cli beautification#254

Open
joelostblom wants to merge 9 commits intomainfrom
refactor/move-cli-beautification
Open

refactor/move cli beautification#254
joelostblom wants to merge 9 commits intomainfrom
refactor/move-cli-beautification

Conversation

@joelostblom
Copy link
Copy Markdown
Contributor

@joelostblom joelostblom commented Apr 2, 2026

Description

Move common functionality to flower. Review together with seedcase-project/seedcase-soil#5 and seedcase-project/check-datapackage#323. More comments in the soil PR.

Needs a thorough review.

Checklist

  • Ran just run-all

pyproject.toml Outdated
]

[tool.uv.sources]
seedcase-soil = { path = "../seedcase-soil", editable = true }
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This need to change before merging; so we wait for the first release of soil on pypi

@joelostblom joelostblom moved this from Todo to In Review in Product development Apr 2, 2026
]

[tool.uv.sources]
seedcase-soil = { git = "https://github.com/seedcase-project/seedcase-soil", rev = "feat/beautify-cli" }
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Wait for a published version of soil before merging so that we can update this

@joelostblom joelostblom marked this pull request as ready for review April 2, 2026 14:23
@joelostblom joelostblom requested a review from a team as a code owner April 2, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

1 participant