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

Release 3.0.0: for CakePHP 4.x #41

Merged
merged 50 commits into from
May 21, 2020
Merged

Release 3.0.0: for CakePHP 4.x #41

merged 50 commits into from
May 21, 2020

Conversation

o0h
Copy link
Contributor

@o0h o0h commented May 17, 2020

This PR is a working branch in preparation for the release.
For specific changes, please see milestone:3.x.
(This milestone will be closed after the 3.x branch has been merged into the master)

letstalk-nl and others added 30 commits January 19, 2020 12:41
# Conflicts:
#	.gitignore
#	README.md
#	composer.json
#	src/Error/SentryErrorHandlerTrait.php
#	src/Plugin.php
… isolation, we do not need the testapp anymore
… Bugfixes have already been part of CakePHP 4.0.0
Re implements test-app in 3.x
Instead of the ErrorLogger::log() that the plugin has, cakephp's default ErrorLogger::log() will be called directly if php's error_handler called directly.
To avoid this, done`Log::setConfig()`with plugin's config before call  ErrorHandler::register()
@codecov-io
Copy link

codecov-io commented May 17, 2020

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #41   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        17        17           
===========================================
  Files              4         4           
  Lines             54        54           
===========================================
  Hits              54        54           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e60c92c...e60c92c. Read the comment docs.

@o0h o0h marked this pull request as ready for review May 21, 2020 06:21
@codecov-commenter
Copy link

codecov-commenter commented May 21, 2020

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##              master       #41   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        17        17           
===========================================
  Files              4         4           
  Lines             54        54           
===========================================
  Hits              54        54           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0681ac2...0681ac2. Read the comment docs.

o0h added 2 commits May 21, 2020 15:52
Only ErrorHandlerTrait had a prefix, so for consistency, removed the prefix "Sentry" like the other classes.
@o0h o0h merged commit a957b4d into master May 21, 2020
@o0h o0h deleted the 3.x branch May 21, 2020 07:00
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.

None yet

5 participants