Merge ~cjwatson/launchpad:expire-questions-limit into launchpad:master
Proposed by
Colin Watson
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | 67a110cfbc858ed7ff7d3de656499f56e9f1daef |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~cjwatson/launchpad:expire-questions-limit |
Merge into: | launchpad:master |
Diff against target: |
125 lines (+43/-5) 4 files modified
cronscripts/expire-questions.py (+13/-1) lib/lp/answers/model/question.py (+5/-2) lib/lp/answers/model/tests/test_question.py (+21/-0) lib/lp/answers/scripts/questionexpiration.py (+4/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ines Almeida | Approve | ||
Review via email: mp+441559@code.launchpad.net |
Commit message
expire-questions: Add a --limit option
Description of the change
This is in line with the otherwise quite similar `expire-bugtasks` script. We currently have a cowboyed patch on dogfood limiting expiry there to five questions for testing; this allows us to do that using a command-line option instead, and to drop that patch.
To post a comment you must log in.
Looks good to me!