Add purge command

[passkey1510]

I think it could be helpful if the bundle has a command to purge a queue. I mainly need it in my unit tests as well as in deployment process to staging & preprod servers. In fact, I had to issue a purge command manually every time I deploy the application to these environments (to have a fresh instance running after each deployment).

[blaugueux]

I suggest to have a confirmation before executing the purge. This can be a critical mistake if you purge in prod.

[videlalvaro]

If we have a confirmation, there should be a way to override it, for sysadmins

[blaugueux]

Maybe with an optional parameter --no-confirmation?

[videlalvaro]

ping @passkey1510

[passkey1510]

Hi there, I was lost in my GitHub notifications :). I've just introduced the "no-confirmation" option in the command. Tell me if it looks good for you guys.

[blaugueux]

I think it's better with the confirmation. Can you squash your commits so @videlalvaro can merge your PR easily.

[passkey1510]

The commits have been squashed!

[blaugueux]

I suggest to add unit tests with this feature. We've experienced some issues recently with the bundle and i think it would be a good practice to add tests with every new feature.

[passkey1510]

Just added an unit test for purge command

[blaugueux]

Looks like good for me but i can't see the travis build. @videlalvaro have you configure the service hook?

[passkey1510]

ping @videlalvaro

[stof]

This phpdoc is describing the class, not the method. So it should be at the class level

[evillemez]

Any news on this? Would love to have an easy way to purge, would help with testing.

[tyx]

👍

[ch3ric]

👍

[videlalvaro]

Sorry for missing this one. I think the missing step would be to document the feature on the README and then I hit the green button

[ch3ric]

Thanks @videlalvaro!
@passkey1510 ping passkey1510#1

[passkey1510]

Hi @ch3ric and @videlalvaro, I've just merge the PR to update the README file. Please considering merging this feature :)

[videlalvaro]

@passkey1510 according to github there are merge conflicts

[passkey1510]

@videlalvaro I've made a rebase on master, could you review this please?

[ch3ric]

thanks @passkey1510 & @videlalvaro