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

Update sfTask.class.php #50

Merged
merged 1 commit into from
Apr 30, 2014
Merged

Conversation

JLepeltier
Copy link
Contributor

Remove deprecated preg_replace with 'e' pattern modifier on sfTask/getDetailedDescription method

@j0k3r
Copy link
Contributor

j0k3r commented Mar 19, 2014

Hey !
It will be awesome if we can have a little test for that in sfTaskTest :)

@JLepeltier
Copy link
Contributor Author

Do you use php 5.5 by now at L'express ?

@j0k3r
Copy link
Contributor

j0k3r commented Mar 19, 2014

Not with symfony1.

@j0k3r
Copy link
Contributor

j0k3r commented Mar 20, 2014

Can you squash your commit? Then I'll merge it.
Thanks

@j0k3r
Copy link
Contributor

j0k3r commented Apr 29, 2014

Ping @JLepeltier ? :)

Remove deprecated preg_replace with 'e' pattern modifier on sfTask/getDetailedDescription method

Update sfTaskTest.php

Add sfTask/getDetailedDescription test

Update sfTaskTest.php

Update sfTask.class.php
@JLepeltier
Copy link
Contributor Author

Sorry, I don't now how to squash with the github web interface :(
It's easyer using CLI :p

2014-04-29 12:23 GMT+02:00 Jeremy notifications@github.com:

Ping @JLepeltier https://github.com/JLepeltier ? :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/50#issuecomment-41660912
.

Jérôme Lepeltier
GG

@j0k3r
Copy link
Contributor

j0k3r commented Apr 30, 2014

Well, you cannot use the Github interface :) You should do it manually.

GromNaN added a commit that referenced this pull request Apr 30, 2014
@GromNaN GromNaN merged commit c542454 into FriendsOfSymfony1:master Apr 30, 2014
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.

3 participants