shortlist() issues UserWarnings rather than raising soft OOPSes
Bug #708099 reported by
Gavin Panella
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Triaged
|
High
|
Unassigned | ||
Bug Description
The checkwatches logs are littered with warnings like:
shortlist() should not be used here. It's meant to listify
sequences with no more than 15 items. There were 40 items.
return self.getConjoin
It would be more useful to generate a soft OOPS report in these cases - they get collated and reported to developers.
| description: | updated |
| description: | updated |
Revision history for this message
| Robert Collins (lifeless) wrote Re: [Bug 708099] Re: shortlist() should accept a context argument | #1 |
Revision history for this message
| Gavin Panella (allenap) wrote Re: shortlist() should accept a context argument | #2 |
Revision history for this message
| Robert Collins (lifeless) wrote Re: [Bug 708099] Re: shortlist() should accept a context argument | #3 |
| summary: |
- shortlist() should accept a context argument + shortlist() issues UserWarnings rather than raising soft OOPSes |
| description: | updated |
| Changed in launchpad: | |
| importance: | Low → High |
| tags: | added: easy |
| tags: | added: oops-infrastructure |
To post a comment you must log in.
Perhaps we should just emit an OOPS rather than emit a warning.
warnings go to logs but skip a lot of context. oopses give us
backtraces, urls etc etc.