Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

code rot review #203

Open
2 tasks
slivingston opened this issue Jun 24, 2018 · 0 comments
Open
2 tasks

code rot review #203

slivingston opened this issue Jun 24, 2018 · 0 comments

Comments

@slivingston
Copy link
Member

Some parts of TuLiP are rarely or never used. In general having good test coverage can prevent rarely used code from unexpectedly breaking. Nonetheless, maintaining rarely used code as TuLiP evolves is not well motivated. However, because usage of TuLiP in the wild is not well known, it is difficult to decide which parts should receive priority of attention.

This issue is intended to track reviewing of the sourcetree as it is now, and

  • to decide which parts should be deprecated, if any, and
  • to discover if any parts are currently broken, despite tests passing.

For now and the future, the following protocol is proposed:

  1. add deprecation notices to parts that we, as the developers, do not think are widely used.
  2. create issue to indicate decision, to collect discussion, possibly change original plan of deprecation.
  3. and then allow 1 year (or 6 months?) to pass before removal.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant