Skip to content

Properly escape backlashes in E614_REGEX - #47

Merged
jklein24 merged 2 commits into
lightsparkdev:mainfrom
Polsaker:patch-1
Mar 13, 2025
Merged

Properly escape backlashes in E614_REGEX#47
jklein24 merged 2 commits into
lightsparkdev:mainfrom
Polsaker:patch-1

Conversation

@Polsaker

Copy link
Copy Markdown
Contributor

Using r-strings to avoid the new "invalid escape sequence" SyntaxWarning in Python 3.12

Using r-strings to avoid the new "invalid escape sequence" `SyntaxWarning` in Python 3.12
@jklein24
jklein24 enabled auto-merge (squash) March 13, 2025 18:45
@jklein24
jklein24 disabled auto-merge March 13, 2025 23:54
@jklein24
jklein24 merged commit 8053c92 into lightsparkdev:main Mar 13, 2025
markatto added a commit that referenced this pull request Jul 22, 2026
setup.py was deleted in #70 but the release workflow still runs `uv run
--frozen python setup.py sdist`, so the next release would fail at the
build step. The v2.10.2 release already failed to publish (its workflow
run predates #70 and died in the old pipenv setup), which is why PyPI is
still at 2.10.1. `uv build` produces the sdist and wheel from
pyproject.toml directly, and `gh-action-pypi-publish` picks them up from
`dist/` as before.

Tested by running `uv build` in this branch: both artifacts build, and
the wheel's `lightspark_client.py` compiles with no SyntaxWarning (the
E614_REGEX escape fix from #47 is included, which 2.10.1 on PyPI
predates).
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