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

Update textual-dev, .gitignore .lock file, update pyproject.toml to latest poetry conventions #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ssteinerx
Copy link

The specified textual dependency was keeping textual waaaaaay back. I changed the dependency to be on textual-dev, which will pull textual along with it so textual doesn't get stuck. Since textual's versions are 0.xx.0, once you pick an xx it will never update due to the way Poetry handles semver.

Added poetry.lock to .gitignore. In my experience, during development (e.g. before 1.0), it just holds things back unnecessarily and leads to dependencies on out of date libraries. Once you hit 1.0, sure, to make sure people are running the right mix, but before that it just makes for confusing conflicts with no real benefit and nasty surprises when people eventually install "the latest" version of something and breaks the world. I'd rather have that happen during development.

I also added ruff since I prefer it for formatting. It's a dev only dependency, so it won't fatten any distribution.

Also, moved the dev dependencies to Poetry's new location of [tool.poetry.group.dev.dependencies] where poetry install --group dev, the "new, improved" way of specifying dev dependencies will put it.

The former textual = "^0.37.1" spec, was keeping textual back at 0.37.?? even though textual-dev is good to 0.60.xxx.

FIX: Only specify textual-dev,  remove explicit textual version spec.

Now, textual-dev is updated, it will pull textual along with it to the latest compatible version.  Look Ma, no hands!

Per Poetry doc: "An update is allowed if the new version number does not modify the left-most non-zero digit in the major, minor, patch grouping."
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.

1 participant