Skip to content

Move _package_version to travertino#3183

Merged
freakboy3742 merged 4 commits into
beeware:mainfrom
mhsmith:travertino-package-version
Feb 13, 2025
Merged

Move _package_version to travertino#3183
freakboy3742 merged 4 commits into
beeware:mainfrom
mhsmith:travertino-package-version

Conversation

@mhsmith

@mhsmith mhsmith commented Feb 11, 2025

Copy link
Copy Markdown
Member

Now that Toga is pinned to a specific version of Travertino, we can consolidate the two copies of this code.

@mhsmith mhsmith force-pushed the travertino-package-version branch from d32fc7c to 615251d Compare February 11, 2025 16:04
@mhsmith mhsmith requested a review from freakboy3742 February 11, 2025 16:28

@freakboy3742 freakboy3742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One comment inline about references to toga inside travertino.

The implicit imports in toga-cocoa/__init__.py et al should also be updated, so we're importing from a canonical location.

Comment thread travertino/src/travertino/__init__.py Outdated
Comment thread core/src/toga/__init__.py Outdated
mhsmith and others added 2 commits February 12, 2025 12:43
@mhsmith mhsmith requested a review from freakboy3742 February 12, 2025 13:08

@freakboy3742 freakboy3742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Missed one usage in toga-dummy (which I've added), but otherwise looks good.

@freakboy3742 freakboy3742 merged commit 564c794 into beeware:main Feb 13, 2025
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