Ticket #6663 (closed: duplicate)
SANSUtility - add deprecate warning line to all its methods
| Reported by: | Gesner Passos | Owned by: | Peter Parker |
|---|---|---|---|
| Priority: | major | Milestone: | Release 3.2 |
| Component: | SANS | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Jay Rainey |
Description (last modified by Gesner Passos) (diff)
Add a deprecate warning line to all the method in this python file.
Gesner found that none of the ISIS sans uses methods from this class but e.g. Richard or Steve may done in some of there script.
Adding the warning now means that later we will be less worried about removing this .py file entirely
Change History
comment:1 Changed 8 years ago by Anders Markvardsen
- Priority changed from minor to major
- Owner changed from Anders Markvardsen to Gesner Passos
- Status changed from new to assigned
- Description modified (diff)
- Summary changed from SANSUtility - never used lines of codes? to SANSUtility - add deprecate warning line to all its methods
comment:5 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:6 Changed 7 years ago by Gesner Passos
- Component changed from Framework to SANS
- Description modified (diff)
Recently, I have added some methods. We'd better add the deprecated warning to the methods that are not being used and start using better that place to put codes that are being duplicated around the SANS reduction scripts.
comment:7 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move to assigned at the introduction of the triage step
comment:9 Changed 7 years ago by Nick Draper
- Owner changed from Gesner Passos to Peter Parker
Ownership handed over to Peter
comment:10 Changed 6 years ago by Peter Parker
- Status changed from assigned to verify
- Resolution set to duplicate
Did this and more as part of #9476. Closing as duplicate.
comment:12 Changed 6 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:13 Changed 6 years ago by Jay Rainey
- Status changed from verifying to closed
Although I tested the ticket that solved this issue (#9476). I built master locally and re-tested it to ensure this deprecated warning was output, which it is. Closing.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7509
