Skip to content

Conversation

@da-viper
Copy link
Contributor

@da-viper da-viper commented Jan 6, 2026

Replace version matching with the new decorator to prevent typos, and make to it clear why we skipped the test.

… test requirements

Replace version matching with the new decorator to prevent typos,
and make to it clear why we skipped the test.
@da-viper da-viper requested a review from ashgti January 6, 2026 00:16
@da-viper da-viper requested a review from JDevlieghere as a code owner January 6, 2026 00:16
@llvmbot llvmbot added the lldb label Jan 6, 2026
@llvmbot
Copy link
Member

llvmbot commented Jan 6, 2026

@llvm/pr-subscribers-lldb

Author: Ebuka Ezike (da-viper)

Changes

Replace version matching with the new decorator to prevent typos, and make to it clear why we skipped the test.


Full diff: https://github.com/llvm/llvm-project/pull/174509.diff

2 Files Affected:

  • (modified) lldb/packages/Python/lldbsuite/test/decorators.py (+8)
  • (modified) lldb/test/API/tools/lldb-dap/launch/TestDAP_launch.py (+8-8)
<html><body><h1>504 Gateway Time-out</h1>
The server didn't respond in time.
</body></html>

@da-viper da-viper merged commit ff20747 into llvm:main Jan 6, 2026
12 checks passed
@da-viper da-viper deleted the add-decorator-conpty branch January 6, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants