Skip to content

Add the $customSamplingContext argument to Hub::startTransaction() and HubAdapter::startTransaction() - #1176

Merged
Jean85 merged 2 commits into
getsentry:masterfrom
ste93cry:fix/add-custom-sampling-context-argument-to-start-transaction-method
Jan 21, 2021
Merged

Jean85 merged 2 commits into
getsentry:masterfrom
ste93cry:fix/add-custom-sampling-context-argument-to-start-transaction-method

Conversation

@ste93cry

Copy link
Copy Markdown
Contributor

As per title: this should fix the deprecation warnings thrown by Symfony's DebugClassLoader and reported in getsentry/sentry-symfony#419

@ruudk ruudk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Much better 👏

@Jean85 Jean85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good job with adding the test too!

@Jean85
Jean85 merged commit 540fd64 into getsentry:master Jan 21, 2021
@ste93cry
ste93cry deleted the fix/add-custom-sampling-context-argument-to-start-transaction-method branch January 21, 2021 20:48
@ruudk

ruudk commented Jan 22, 2021

Copy link
Copy Markdown

Thanks for the fast fix on this @ste93cry. Would it be too much if I asked for a tagged release of this? Would help me a lot. Thanks 🙏

@ste93cry

Copy link
Copy Markdown
Contributor Author

There is at least another bugfix that I want to merge before releasing the next patch version. I will try to see if I can merge it during the weekend and prepare the release for the first days of the next week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants