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

Add symfony 3.0 support #202

Merged
merged 12 commits into from
Feb 3, 2016
Merged

Add symfony 3.0 support #202

merged 12 commits into from
Feb 3, 2016

Conversation

GuilhemN
Copy link
Collaborator

@GuilhemN GuilhemN commented Jan 5, 2016

Replaces #198

Changes done on top of that PR:

  • Inject security.context in older versions
  • Use better composer.json version constraints
  • Update Travis configuration and test matrix

@GuilhemN GuilhemN changed the title Add symfony 3.0 support [WIP] Add symfony 3.0 support Jan 5, 2016
@thomask thomask mentioned this pull request Jan 6, 2016
@GuilhemN
Copy link
Collaborator Author

GuilhemN commented Jan 6, 2016

Any idea why the annotations are not loaded in the functional tests?

@GuilhemN
Copy link
Collaborator Author

GuilhemN commented Jan 6, 2016

It seems to be fixed by loading them in the AppKernel manually. I think this was caused by the @runInSeparateProcess annotation.

@GuilhemN GuilhemN changed the title [WIP] Add symfony 3.0 support Add symfony 3.0 support Jan 10, 2016
@GuilhemN
Copy link
Collaborator Author

The tests are completely fixed this time!

@GuilhemN
Copy link
Collaborator Author

Can you take a look @schmittjoh please?

@Ninir
Copy link
Contributor

Ninir commented Jan 11, 2016

Good job @Ener-Getick :-)
Would be awesome to have it by this week!

@heikokrebs
Copy link

Would be great! :)

break;
}
$dir = dirname($dir);
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO, this logic is useless. Tests are not meant to be run with the bundle being the root package. Otherwise, you won't have the optional deps

@Jean85
Copy link
Contributor

Jean85 commented Jan 11, 2016

I agree with what @stof said.
Otherwise, 👍 good work!!

@whyte624
Copy link

👍

@pinouf
Copy link

pinouf commented Jan 12, 2016

very good -)
Could we get a merge -) ?

@e-doceo
Copy link

e-doceo commented Jan 12, 2016

+1

@kYem
Copy link

kYem commented Jan 14, 2016

👍

1 similar comment
@j4k4
Copy link

j4k4 commented Jan 14, 2016

👍

@lenar
Copy link

lenar commented Jan 17, 2016

👍

@Jean85
Copy link
Contributor

Jean85 commented Jan 18, 2016

@schmittjoh can we merge this? We're all waiting for it! 🎱

@GuilhemN
Copy link
Collaborator Author

@Jean85 as a temporary solution you can use my repository (see https://getcomposer.org/doc/05-repositories.md#vcs) with dev-SF3 ;-)

But be aware that I will probably remove it when this PR will be merged.

@heikokrebs
Copy link

@schmittjoh Any plans about merging this PR? Would help a lot ;)

@kYem
Copy link

kYem commented Jan 25, 2016

Ye, would definitely help, as this is one of the few remaining issues, stopping us from upgrading to 3.0. If there is anything we could do speed up the merge, let us know.

@GuilhemN
Copy link
Collaborator Author

@kYem you can try to ping @schmittjoh on twitter ;-)

@sblondeau
Copy link

👍 Thx

@Catalin-Ionut
Copy link

@schmittjoh could we please have this merged? alot of projects await to upgrade to symfony 3.0

@FredoVelcro-zz
Copy link

+1

3 similar comments
@pesu
Copy link

pesu commented Feb 1, 2016

+1

@giovkanata
Copy link

+1

@pculka
Copy link

pculka commented Feb 2, 2016

+1

@heikokrebs
Copy link

@schmittjoh ping :)

GuilhemN added a commit that referenced this pull request Feb 3, 2016
@GuilhemN GuilhemN merged commit 527cff1 into schmittjoh:master Feb 3, 2016
@GuilhemN GuilhemN deleted the SF3 branch February 3, 2016 12:23
@GuilhemN
Copy link
Collaborator Author

GuilhemN commented Feb 3, 2016

Finally merged :-)
I'll create an RC phase if you could test it on your projects it would be awesome !

@pculka
Copy link

pculka commented Feb 3, 2016

Kudos @Ener-Getick & @schmittjoh !

@GuilhemN
Copy link
Collaborator Author

GuilhemN commented Feb 3, 2016

@Jean85
Copy link
Contributor

Jean85 commented Feb 3, 2016

Woha!!! Great!!
Congrats @Ener-Getick on becoming a mantainer! ;)

@GuilhemN
Copy link
Collaborator Author

GuilhemN commented Feb 3, 2016

Thank you @Jean85 :-)

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.