#7534 closed enhancement (fixed)
Add a deprecation message at the top of most sage.server.* files
Reported by: | mpatel | Owned by: | boothby |
---|---|---|---|
Priority: | minor | Milestone: | sage-4.3 |
Component: | notebook | Keywords: | |
Cc: | timdumol, was | Merged in: | sage-4.3.rc1 |
Authors: | Tim Dumol | Reviewers: | Mike Hansen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
To avoid confusion, we should add a message to the top of each old Sage notebook .py file stating that one should work on SageNB instead.
This should be a patch to the core Sage library.
Attachments (2)
Change History (12)
comment:1 Changed 11 years ago by
- Cc timdumol was added
comment:2 Changed 11 years ago by
See sage-notebook.
comment:4 Changed 11 years ago by
- Reviewers set to Mike Hansen
- Status changed from needs_review to needs_work
This causes some failures in tests in sage/server/
comment:5 follow-up: ↓ 6 Changed 11 years ago by
{{{ sage -t devel/sage-main/sage/server/notebook/notebook_object.py # 4 doctests failed }}}
comment:6 in reply to: ↑ 5 Changed 11 years ago by
Replying to mhansen:
{{{ sage -t devel/sage-main/sage/server/notebook/notebook_object.py # 4 doctests failed }}}
Since the code isn't being run and is officially "deprecated", one option is to put a nodoctest.py file in the directory (or #nodoctest at the top of the file) so that the code isn't tested.
comment:7 Changed 11 years ago by
- Status changed from needs_work to needs_review
V2 suppresses doctesting for the "deprecated" files.
comment:8 Changed 11 years ago by
- Status changed from needs_review to positive_review
Looks good to me now.
comment:9 Changed 11 years ago by
- Merged in set to sage-4.3.rc1
- Resolution set to fixed
- Status changed from positive_review to closed
comment:10 Changed 11 years ago by
- Milestone set to sage-4.3
How about
?