Skip to content

Conversation

@matawed
Copy link

@matawed matawed commented Aug 6, 2025

Description

I know that this project doesn't claim to be compatible with Python 3.12, but since I'm using this project on a machine with this setup and I get the following warnings:

/tmp/Bazel.runfiles_4bm23prw/runfiles/rules_python++pip+athenacli_requirements_312_athenacli_py2_none_any_adafebae/site-packages/athenacli/main.py:207: SyntaxWarning: invalid escape sequence '\e'
  """
/tmp/Bazel.runfiles_4bm23prw/runfiles/rules_python++pip+athenacli_requirements_312_athenacli_py2_none_any_adafebae/site-packages/athenacli/packages/special/iocommands.py:120: SyntaxWarning: invalid escape sequence '\e'
  pattern = re.compile('(^\\\e|\\\e$)')
/tmp/Bazel.runfiles_4bm23prw/runfiles/rules_python++pip+athenacli_requirements_312_athenacli_py2_none_any_adafebae/site-packages/athenacli/packages/special/iocommands.py:216: SyntaxWarning: invalid escape sequence '\d'
  match = re.search('\\$\d+', query)
/tmp/Bazel.runfiles_4bm23prw/runfiles/rules_python++pip+athenacli_requirements_312_athenacli_py2_none_any_adafebae/site-packages/athenacli/packages/parseutils.py:18: SyntaxWarning: invalid escape sequence '\d'
  """
/tmp/Bazel.runfiles_4bm23prw/runfiles/rules_python++pip+athenacli_requirements_312_athenacli_py2_none_any_adafebae/site-packages/athenacli/completer.py:31: SyntaxWarning: invalid escape sequence '\$'

I thought this PR might be a quick fix for that.

Checklist

  • I've added this contribution to the changelog.md.
  • I've added my name to the AUTHORS file (or it's already there).

@matawed matawed force-pushed the mt/fix_warnings_related_to_escape_seqs_py_312 branch from aabe9e6 to edeebdc Compare August 6, 2025 07:00
@matawed matawed changed the title fix: use r in front of strings with escape seqs fix: use raw strings to prevent warnings about escape sequences in strings Aug 6, 2025
@matawed matawed marked this pull request as ready for review August 6, 2025 10:01
@matawed matawed force-pushed the mt/fix_warnings_related_to_escape_seqs_py_312 branch from edeebdc to 9e0f159 Compare August 8, 2025 13:42
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