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

Fix doctrine/inflector deprecations and allow to install v2 #963

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

jdreesen
Copy link

Fixes #962 (at least for v6.2, not sure what your version branch merge strategy is...)

@codecov-commenter
Copy link

codecov-commenter commented Aug 26, 2020

Codecov Report

Merging #963 into 6.2 will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##                6.2     #963   +/-   ##
=========================================
  Coverage     66.61%   66.61%           
  Complexity      423      423           
=========================================
  Files            39       39           
  Lines          1393     1393           
=========================================
  Hits            928      928           
  Misses          465      465           
Impacted Files Coverage Δ Complexity Δ
Mapping/Caser.php 100.00% <100.00%> (ø) 2.00 <0.00> (ø)

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 ac771ff...efb85ed. Read the comment docs.

Mapping/Caser.php Outdated Show resolved Hide resolved
@jdreesen jdreesen force-pushed the fix-doctrine-inflector-deprecations branch from efb85ed to 043a7ec Compare August 31, 2020 12:43
@pkly
Copy link
Contributor

pkly commented Dec 7, 2020

@saimaz @einorler could one of you possibly merge this and release a new version?

@einorler einorler merged commit 6fe42dd into ongr-io:6.2 Dec 7, 2020
@jdreesen jdreesen deleted the fix-doctrine-inflector-deprecations branch December 7, 2020 10:04
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.

4 participants