Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Display locals in traceback when requested #2900

Merged
merged 2 commits into from
May 30, 2024
Merged

Conversation

happz
Copy link
Collaborator

@happz happz commented May 2, 2024

By setting TMT_SHOW_TRACEBACK to full, tmt would dump locals for each frame.

Pull Request Checklist

  • implement the feature

@happz happz added the code | logging Changes related to debug logging label May 2, 2024
@happz happz added this to the 1.34 milestone May 2, 2024
@happz
Copy link
Collaborator Author

happz commented May 2, 2024

2024-05-02_22-05

@happz happz added the ci | full test Pull request is ready for the full test execution label May 16, 2024
@happz happz force-pushed the traceback-show-locals branch 3 times, most recently from 56377a9 to d154291 Compare May 21, 2024 09:07
Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, just docstring?

tmt/utils.py Show resolved Hide resolved
@happz happz force-pushed the traceback-show-locals branch from d154291 to 71a3cab Compare May 30, 2024 11:57
@psss psss added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label May 30, 2024
happz added 2 commits May 30, 2024 15:28
By setting `TMT_SHOW_TRACEBACK` to `full`, tmt would dump locals for
each frame.
@happz happz force-pushed the traceback-show-locals branch from 71a3cab to 4dfae6c Compare May 30, 2024 13:29
@happz happz enabled auto-merge (squash) May 30, 2024 13:29
@happz happz merged commit 763ab00 into main May 30, 2024
18 of 19 checks passed
@happz happz deleted the traceback-show-locals branch May 30, 2024 16:20
happz added a commit that referenced this pull request Jun 2, 2024
By setting `TMT_SHOW_TRACEBACK` to `full`, tmt would dump locals for
each frame.
The-Mule pushed a commit to The-Mule/tmt that referenced this pull request Oct 14, 2024
By setting `TMT_SHOW_TRACEBACK` to `full`, tmt would dump locals for
each frame.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution code | logging Changes related to debug logging status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants