Skip to content

Be aware of deprecations #145

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

Merged
merged 7 commits into from
Nov 1, 2017
Merged

Conversation

ElectricMaxxx
Copy link
Member

Q A
Branch? "master" for new features / the branch of the current release for fixes
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets
License MIT

This code helps our other bundles to avoid lots of deprecation warnings.

@ElectricMaxxx
Copy link
Member Author

@xabbuh @derrabus i think that should be the way not to get the deprecation warnings of services, which became private now. At the end we should make them public on doctrine/phpcr-bundle but this one works for now.

@ElectricMaxxx
Copy link
Member Author

Maybe we do not need to make the services public here, when i found them all in doctrine/DoctrinePHPCRBundle#286

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests
@ElectricMaxxx ElectricMaxxx force-pushed the be_aware_of_deprecations branch from 8efffaf to 4eb229d Compare November 1, 2017 06:32
@ElectricMaxxx
Copy link
Member Author

i merge this on into the dev-kit branch not to go over two branches anymore (confuses me g)

@ElectricMaxxx ElectricMaxxx merged commit 754eda9 into master-dev-kit Nov 1, 2017
@ElectricMaxxx ElectricMaxxx deleted the be_aware_of_deprecations branch November 1, 2017 06:33
ElectricMaxxx added a commit that referenced this pull request Nov 3, 2017
* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public
ElectricMaxxx added a commit that referenced this pull request Nov 3, 2017
* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public
ElectricMaxxx added a commit that referenced this pull request Nov 4, 2017
* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public
ElectricMaxxx added a commit that referenced this pull request Nov 4, 2017
* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public
ElectricMaxxx added a commit that referenced this pull request Nov 8, 2017
introduce KENEL_CLASS handling, usage of newer phpcr-bundle with correct
service visibility

merge conflicts

Be aware of deprecations (#145)

* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public

add yaml component

merge conflicts
ElectricMaxxx added a commit that referenced this pull request Nov 8, 2017
* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public

add minimum to composer.json

(cherry picked from commit e4c4bc5)

add yaml component

move logs and cache dir

use newest phpcr-bundle now

fix version increase to increas phpcr-bundle (#149)

DevKit updates

add change log entry

apply changes from styleCI
ElectricMaxxx added a commit that referenced this pull request Nov 8, 2017
* be aware of deprectations

introduce KENEL_CLASS handling, usage of newer phpcr-bundle with correct
service visibility

merge conflicts

Be aware of deprecations (#145)

* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public

add yaml component

merge conflicts

* Be aware of deprecations (#145)

* merge conflicts

* merge conflicts

* make some services public and do not use KERNEL_DIR

change when exist

use new KERNEL_CLASS env var

fixes due to style hints

revert service visibility change

revert it

increase symfony versions

use phpcr-bundle here

revert 4.0 stuff here for now

Merge conflicts

adjust tests

* fixes due to style issue

* without any symfony version

* add a new service to the list to set public

* remove compiler pass to make them public

add minimum to composer.json

(cherry picked from commit e4c4bc5)

add yaml component

move logs and cache dir

use newest phpcr-bundle now

fix version increase to increas phpcr-bundle (#149)

DevKit updates

add change log entry

apply changes from styleCI

* use correct phpcr-bundle version
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.

1 participant