Skip to content

Small cleanups: dead code, version string, Loudmouth regex (#40, #41, #28)#57

Open
dbqpdb wants to merge 3 commits into
masterfrom
chore/small-cleanups
Open

Small cleanups: dead code, version string, Loudmouth regex (#40, #41, #28)#57
dbqpdb wants to merge 3 commits into
masterfrom
chore/small-cleanups

Conversation

@dbqpdb

@dbqpdb dbqpdb commented May 15, 2026

Copy link
Copy Markdown
Owner

Summary

Three small, independent fixes that don't conflict with anything else in flight. Each is one commit.

Test plan

Why these are bundled

All three are independent of the other six open PRs:

No rebase friction expected against any of the existing PRs.

Closes #40, #41, #28.

🤖 Generated with Claude Code

dbqpdb and others added 3 commits May 14, 2026 20:25
- Removed sillyDude() — was never called.
- Removed run_him_if_i_am_not_the_main_file() and the trailing else
  branch that called it on every import; importing rlm is now silent
  (which the test suite already assumed).

`runFromOutside.py` is the third item in #40 but is already covered
by #49 (file deletion).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Float versioning breaks ordering (0.10 < 0.9 as floats) and 0.40 == 0.4.
String is PEP 440-compatible and dunder name is convention for module
version.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Two different patterns were being used for the same purpose: '\{.+?\}'
for findall, '\{.*?\}' for sub. Now compiled once as a raw string and
reused — same pattern in both call sites, no more SyntaxWarning about
the invalid \{ escape.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

Cleanup: remove dead code paths

1 participant