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

Revert "feat: profile concurrent transactions (#2105)" #2225

Merged

Conversation

armcknight
Copy link
Member

This reverts commit e43ce74.

Found an issue involving either automatic UI profiling or child spans, need to investigate.

#skip-changelog

@armcknight armcknight merged commit 4c1fab4 into master Sep 26, 2022
@armcknight armcknight deleted the armcknight/revert-profiling-concurrent-transactions branch September 26, 2022 20:52
armcknight added a commit that referenced this pull request Sep 26, 2022
philipphofmann added a commit that referenced this pull request Sep 27, 2022
philipphofmann added a commit that referenced this pull request Sep 27, 2022
kevinrenskers added a commit that referenced this pull request Sep 27, 2022
* master:
  release: 7.26.0
  meta: Fix Changelog concurrent transactions (#2229)
  build(deps): bump fastlane from 2.210.0 to 2.210.1 (#2224)
  Revert "feat: profile concurrent transactions (#2105)" (#2225)
  fix: Align core data span operations (#2222)
  test: Remove empty assert msg for AppStateTests (#2221)
  meta: Fix Changelog (#2219)
  ref: Add typealias for bytes (#2209)
  feat: profile concurrent transactions (#2105)
  ci: Readd cache for UI tests (#2215)

# Conflicts:
#	Sentry.xcodeproj/project.pbxproj
armcknight added a commit that referenced this pull request Oct 25, 2022
armcknight added a commit that referenced this pull request Oct 25, 2022
armcknight added a commit that referenced this pull request Oct 25, 2022
armcknight added a commit that referenced this pull request Oct 25, 2022
* Revert "Revert "feat: profile concurrent transactions (#2105)" (#2225)"

This reverts commit 4c1fab4.

* fix tests and test sdk logging
* provide default for no build numbers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants