Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

textcode markup: fix SyntaxWarning in regex #3891

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

gotmax23
Copy link
Contributor

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Signed-off-by: Maxwell G <maxwell@gtmx.me>
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@pombredanne
Copy link
Member

Merging and ignoring Azure Ci heisenbug:

              ERROR: Unknown error:
E               Traceback (most recent call last):
E                 File "/home/vsts/work/1/s/src/scancode/interrupt.py", line 89, in interruptible
E                   create_signal(SIGALRM, handler)
E                 File "/opt/hostedtoolcache/Python/3.12.6/x64/lib/python3.12/signal.py", line 58, in signal
E                   handler = _signal.signal(_enum_to_int(signalnum), _enum_to_int(handler))
E                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E               ValueError: signal only works in main thread of the main interpreter

@pombredanne pombredanne merged commit 5d50052 into aboutcode-org:develop Oct 4, 2024
35 of 37 checks passed
@gotmax23
Copy link
Contributor Author

gotmax23 commented Oct 5, 2024

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants