Skip to content

Fix: python313 invalid escape sequences - #70

Closed
hdd wants to merge 2 commits into
mainfrom
fix/python313-invalid-escape-sequences
Closed

hdd wants to merge 2 commits into
mainfrom
fix/python313-invalid-escape-sequences

Conversation

@hdd

@hdd hdd commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes #68

Two SyntaxWarning issues on Python 3.13 caused by invalid escape sequences:

  • source/ftrack_api/plugin.py: added r prefix to the discover docstring to suppress warnings for * and ** (RST
    markup for *args/**kwargs)
  • source/ftrack_api/structure/standard.py: added r prefix to the re.sub regex pattern to suppress warning for \w

No functional changes.

Summary by CodeRabbit

  • Chores
    • Made internal string patterns clearer without changing behavior.

hdd and others added 2 commits September 24, 2026 17:30
… regex (#68)

Add r-prefix to regex string in StandardStructure.sanitise_for_filesystem
to fix SyntaxWarning on Python 3.13.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… docstring (#68)

Add r-prefix to docstring in plugin.discover to fix SyntaxWarning on
Python 3.13 caused by \* and \*\* RST markup being treated as invalid
escape sequences.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hdd
hdd requested a review from dennisweil September 24, 2026 15:34
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 889e1f5b-272d-449e-9ec1-475225aa1dd3

📥 Commits

Reviewing files that changed from the base of the PR and between 4ee2ffb and 583306a.

📒 Files selected for processing (2)
  • source/ftrack_api/plugin.py
  • source/ftrack_api/structure/standard.py
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@hdd hdd closed this Sep 24, 2026
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.

1 participant