launch a repairDatabase task on clear_resolved #468

Merged
merged 2 commits into from May 4, 2013

Projects

None yet

3 participants

@shingara
Member

The task db:clear_resolved can delete a lot of data. I think it can be good to repairDatabase after to avoid a big database.

see #458

@coveralls

Coverage Status

Coverage decreased (-0.39%) when pulling 38ca535 on shingara:patch-1 into a761933 on errbit:master.

@coveralls

Coverage Status

Coverage decreased (-0.39%) when pulling 12d9152 on shingara:patch-1 into a761933 on errbit:master.

@coveralls

Coverage Status

Coverage decreased (-0.39%) when pulling 12d9152 on shingara:patch-1 into a761933 on errbit:master.

shingara added some commits Apr 30, 2013
@shingara shingara launch a repairDatabase task on clear_resolved
The task db:clear_resolved can delete a lot of data. I think it can be good to repairDatabase after to avoid a big database.

see #458
ca48b06
@shingara shingara Extract task to clear problem resolved in Interaction class
Extract the task in a class doing the job
Add test on this class
Update rspec and rspec-rails gems
c44f9c1
@shingara
Member
shingara commented May 4, 2013

I think can be merged now. Any objection ?

@coveralls

Coverage Status

Coverage remained the same when pulling c44f9c1 on shingara:patch-1 into 33f2ebc on errbit:master.

@ndbroadbent
Member

Merging, looks good to me!

@ndbroadbent ndbroadbent merged commit 0321d15 into errbit:master May 4, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment