Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Remove trimming from tracing #387

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Remove trimming from tracing #387

wants to merge 8 commits into from

Conversation

ewd3v
Copy link

@ewd3v ewd3v commented Jun 17, 2023

Original code said there was no way to specify a trace level without also specifying a message, but you can just set the message to nil to achieve this.

This change makes it so we don't have to trim the traceback to remove the newline.

Checklist before submitting:

  • Added entry to CHANGELOG.md
  • Added/updated relevant tests
  • Added/updated documentation

ewd3v added 4 commits June 17, 2023 19:03
Original code said there was no to specify a trace level without also specifying a message, but you can just set the message to nil to achieve this.

This change makes it so we don't have to trim the traceback to remove the newline.
@ewd3v ewd3v changed the title Remove trimming from element tracing Remove trimming from tracing Jun 17, 2023
Copy link

@Awokein Awokein left a comment

Choose a reason for hiding this comment

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

Simple but necessary change. Good job 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants