Skip to content

hl2file.py 0.4: Port to Python 3 - #619

Closed
jspricke wants to merge 1 commit into
weechat:mainfrom
jspricke:update_hl2file
Closed

jspricke wants to merge 1 commit into
weechat:mainfrom
jspricke:update_hl2file

Conversation

@jspricke

@jspricke jspricke commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Script info

  • Script name: hl2file.py
  • Version: 0.4

Description

  • Fix and improve code.
  • Format using ruff.

Checklist (script update)

  • Author has been contacted
  • Single commit, single file added
  • Commit message format: script_name.py X.Y: …
  • Script version and Changelog have been updated
  • For Python script: works with Python 3 (Python 2 support is optional)
  • Score 100 / 100 displayed by weechat-script-lint

@jspricke jspricke changed the title ## Script info hl2file.py 0.4: Port to Python 3 Nov 14, 2026
@flashcode flashcode self-assigned this Nov 14, 2026
@flashcode flashcode added bug Unexpected problem or unintended behavior python3 Script is not compatible with Python 3 refactoring Code refactoring labels Nov 14, 2026

@flashcode flashcode 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.

Hi @jspricke,

Thanks for the script update!

I made some comments with changes to do (related to your changes, but also bugs predating your changes), please check them.

Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
@flashcode flashcode added the waiting info Waiting for info from author of issue label Nov 14, 2026
- Fix and improve code.
- Format using ruff.
- Also log notify_private.

@jspricke jspricke left a comment

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.

Thanks for all the comments!

Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved
Comment thread python/hl2file.py Resolved

@flashcode flashcode 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.

Thanks for the quick updates, all good now!

I'm going to merge, and I'll add from __future__ import annotations on my side: the str | list[str] annotation needs Python 3.10 or newer, and some still-supported distributions (like RHEL 9, Rocky and Alma) still ship WeeChat built with Python 3.9.

@flashcode

Copy link
Copy Markdown
Member

Merged in 0798446 (with from __future__ import annotations added, changelog updated and hl2file.py removed from REUSE.toml), thanks!

@flashcode flashcode closed this Nov 14, 2026
@flashcode flashcode removed the waiting info Waiting for info from author of issue label Nov 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Unexpected problem or unintended behavior python3 Script is not compatible with Python 3 refactoring Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants