Skip to content

Release version guard test PR - #27

Closed
Natuworkguy wants to merge 25 commits into
mainfrom
dev
Closed

Release version guard test PR#27
Natuworkguy wants to merge 25 commits into
mainfrom
dev

Conversation

@Natuworkguy

Copy link
Copy Markdown
Owner

No description provided.

Add Text.center(pos) to reposition text so it's centered on a given
point, offsetting by half the rendered width/height and refreshing the
text surface to keep text_rect in sync.
Move the centering math into Entity.center() so all entities get it, and have Text.center() delegate via super() before rebuilding its text surface. Removes the duplicated positioning logic.
Font.render only accepts keyword arguments on pygame-ce, not upstream
pygame, causing a TypeError on systems with vanilla pygame installed. Switch to positional arguments, which are valid on both
variants.

Note that upstream pygame compatibility is not a requirement. The
engine targets pygame-ce. This change is a
low-cost portability improvement, not a supported configuration. (See #6)
@Natuworkguy Natuworkguy added the release Official release pull request. Allows merging directly into main. label Aug 8, 2026

@github-actions github-actions Bot 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.

@Natuworkguy This release PR does not bump engine/version.py (still 0.2.5). Update __version__ before merging.

@github-actions github-actions Bot 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.

@Natuworkguy This release PR does not bump engine/version.py (still 0.2.5). Update __version__ before merging.

@Natuworkguy Natuworkguy closed this Aug 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Official release pull request. Allows merging directly into main.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant