#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 13 years ago by
Cc: | timdumol was added |
---|
Changed 13 years ago by
Attachment: | trac_7534-deprecation-message added |
---|
Adds the recommended deprecation message (with typo correction)
comment:4 Changed 13 years ago by
Authors: | → Tim Dumol |
---|---|
Reviewers: | → Mike Hansen |
Status: | needs_review → needs_work |
This causes some failures in tests in sage/server/
comment:5 follow-up: 6 Changed 13 years ago by
{{{ sage -t devel/sage-main/sage/server/notebook/notebook_object.py # 4 doctests failed }}}
comment:6 Changed 13 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.
Changed 13 years ago by
Attachment: | trac_7534-deprecation-message_v2.patch added |
---|
Adds nodoctest.py
files. Replacement patch.
comment:7 Changed 13 years ago by
Status: | needs_work → needs_review |
---|
V2 suppresses doctesting for the "deprecated" files.
comment:9 Changed 13 years ago by
Merged in: | → sage-4.3.rc1 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
comment:10 Changed 13 years ago by
Milestone: | → sage-4.3 |
---|
How about
?