Skip to content

Conversation

@cleptric
Copy link
Member

@cleptric cleptric commented Oct 6, 2022

Since v3.8.0 https://github.com/getsentry/sentry-php, the SDK NoOps without a DSN set. This fixes the E2E tests.

@cleptric cleptric requested a review from Jean85 October 6, 2022 14:18
@Jean85 Jean85 merged commit 56f78c3 into master Oct 6, 2022
@Jean85 Jean85 deleted the add-dsn-to-e2e branch October 6, 2022 20:12
cleptric added a commit that referenced this pull request Oct 11, 2022
# This is the 1st commit message:

ref: Use constant for the SDK version

# This is the commit message #2:

docs: Update README.md & CONTRIBUTING.md (#650)

* docs: Update README.md & CONTRIBUTING.md

* Add slogan
# This is the commit message #3:

test: Add dsn to e2e test config (#657)

* test: Add dsn to e2e test config

* Add phpstan exclusion
# This is the commit message #4:

Fix Symfony 6 deprecations in E2E test (#656)


# This is the commit message #5:

fix: Update span ops (#655)
cleptric added a commit that referenced this pull request Oct 11, 2022
# This is the 1st commit message:

ref: Use constant for the SDK version

# This is the commit message #2:

docs: Update README.md & CONTRIBUTING.md (#650)

* docs: Update README.md & CONTRIBUTING.md

* Add slogan
# This is the commit message #3:

test: Add dsn to e2e test config (#657)

* test: Add dsn to e2e test config

* Add phpstan exclusion
# This is the commit message #4:

Fix Symfony 6 deprecations in E2E test (#656)

# This is the commit message #5:

fix: Update span ops (#655)

# This is the commit message #6:

docs: Update changelog (#658)
cleptric added a commit that referenced this pull request Oct 11, 2022
docs: Update README.md & CONTRIBUTING.md (#650)

* docs: Update README.md & CONTRIBUTING.md

* Add slogan

test: Add dsn to e2e test config (#657)

* test: Add dsn to e2e test config

* Add phpstan exclusion

Fix Symfony 6 deprecations in E2E test (#656)

fix: Update span ops (#655)

docs: Update changelog (#658)

Prepare release 4.3.1 (#659)

Update phpstan.neon

Add bump-version.sh
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