Skip to content
This repository has been archived by the owner on Jan 17, 2022. It is now read-only.

add suggestion to use pixelfederation/doctrine-resettable-em-bundle #226

Merged
merged 1 commit into from
May 29, 2020

Conversation

Rastusik
Copy link
Contributor

@Rastusik Rastusik commented May 29, 2020

Composer - added suggestion to use pixelfederation/doctrine-resettable-em-bundle to be able to recover from entity manager errors.
This was made according to #185 , no code needed to be changed for the integration

Closes #185

@Rastusik Rastusik force-pushed the 185_resettable_em_bundle branch from d9c1c11 to 9dbfa93 Compare May 29, 2020 13:53
@codecov
Copy link

codecov bot commented May 29, 2020

Codecov Report

Merging #226 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #226   +/-   ##
==========================================
  Coverage      87.23%   87.23%           
  Complexity       617      617           
==========================================
  Files             85       85           
  Lines           1903     1903           
==========================================
  Hits            1660     1660           
  Misses           243      243           

@Rastusik
Copy link
Contributor Author

Rastusik commented May 29, 2020

not sure whether I can do something with the failed check

@k911
Copy link
Owner

k911 commented May 29, 2020

Thanks, @Rastusik I also recommend adding at least two or three sentences in the docs directory, what do you think?

@Rastusik
Copy link
Contributor Author

no problem, will do

…ttable-em-bundle to be able to recover from entity manager errors
@Rastusik Rastusik force-pushed the 185_resettable_em_bundle branch from 9dbfa93 to cf61fdc Compare May 29, 2020 16:13
@codeclimate
Copy link

codeclimate bot commented May 29, 2020

Code Climate has analyzed commit cf61fdc and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 87.2% (0.0% change).

View more on Code Climate.

@k911 k911 merged commit 496c5fe into k911:develop May 29, 2020
@k911 k911 changed the title MR for #185 add suggestion to use pixelfederation/doctrine-resettable-em-bundle May 29, 2020
@k911
Copy link
Owner

k911 commented May 29, 2020

Thanks, @Rastusik, It'll be released soon.

k911 added a commit that referenced this pull request Jun 24, 2020
* build(deps-dev): bump doctrine/annotations from 1.10.2 to 1.10.3 (#223) ([94ef806](94ef806)), closes [#223](#223)
* build(deps-dev): bump phpstan/phpstan from 0.12.28 to 0.12.29 (#249) ([ebf9ce1](ebf9ce1)), closes [#249](#249)
* build(deps-dev): bump phpstan/phpstan-doctrine from 0.12.15 to 0.12.16 (#248) ([830a9ef](830a9ef)), closes [#248](#248)
* build(deps-dev): bump phpstan/phpstan-doctrine from 0.12.16 to 0.12.17 (#263) ([d796528](d796528)), closes [#263](#263)
* chore(composer): add suggestion to use pixelfederation/doctrine-resettable-em-bundle to be able to r ([496c5fe](496c5fe)), closes [#226](#226)
* chore(composer): upgrade deps (#246) ([7e44a89](7e44a89)), closes [#246](#246)
* chore(composer): upgrade deps (#262) ([c9b218a](c9b218a)), closes [#262](#262)
* chore(deps): upgrade composer dependencies (#222) ([37d2517](37d2517)), closes [#222](#222)
* fix(config): allow configuring worker and reactor counts using ENV variables (#244) ([d6b270a](d6b270a)), closes [#244](#244)
* fix(profiler): make log collection in symfony profiler to work (#242) ([50fdd6f](50fdd6f)), closes [#242](#242)
* refactor(error-handler): move to proper namespace (#251) ([4c1c3e9](4c1c3e9)), closes [#251](#251)
* feat(blackfire): Add bridge for upscale/swoole-blackfire (#221) ([960ddb8](960ddb8)), closes [#221](#221)
* feat(exception-handler): Add Symfony error/exception handler (#228) ([180d5e5](180d5e5)), closes [#228](#228)
* feat(http-server): configurable mime types for advanced static files server (#240) ([07896a4](07896a4)), closes [#240](#240)
* ci(circle): upgrade docker client image to v19.03.11 (#239) ([fe7b46c](fe7b46c)), closes [#239](#239)
@Rastusik Rastusik deleted the 185_resettable_em_bundle branch July 27, 2022 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace entity manager handler with doctrine resettable em bundle
2 participants