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

Allow documenting option deprecation #65

Closed
bcoca opened this issue May 31, 2017 · 5 comments
Closed

Allow documenting option deprecation #65

bcoca opened this issue May 31, 2017 · 5 comments

Comments

@bcoca
Copy link
Member

bcoca commented May 31, 2017

Proposal: deprecated doc key for options

Author: Brian Coca <@bcoca> IRC: bcoca

Date: 2017/06/-1

  • Status: New
  • Proposal type: doc fields
  • Targeted Release: 2.4
  • Estimated time to implement: 5mins

Motivation

There is no way to document the deprecation of an option in plugins.

Problems

Will allow authors to document deprecation of options.

Solution proposal

A deprecated key that is composed of the following:

deprecated:
     why: reason for deprecation
     version: version in which this option is removed
     alternatives: description of alternatives to use

Dependencies (optional)

Doc validator needs to permit

@gundalow
Copy link
Contributor

gundalow commented Jun 1, 2017

@gundalow
Copy link
Contributor

gundalow commented Jun 1, 2017

Discussed in Core meeting 1st June 2017

  • +1 from gundalow, sivel, shertel

The above also believe that the module level deprecations should move to using this

Docs also need updating to point out how to deprecate options in argspec

@gundalow
Copy link
Contributor

gundalow commented Jun 6, 2017

Once merged the the following modules will need updating to make use of it:

  • network/f5/bigip_pool.py

Feel free to edit this comment to add other modules to the list

@gundalow
Copy link
Contributor

gundalow commented Jun 8, 2017

Discussed in Core meeting *st June 2017
+1 from Core thaumos ryansb jctanner gundalow, sivel, shertel
+1 from community: dminca

RESULT: Agreed

bcoca may do this as part of the Ansible config project

@bcoca
Copy link
Member Author

bcoca commented Sep 21, 2017

released in 2.4: infrastructure to support it and started using in plugins and config definitions/entries, new module deprecation should use the new format.

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

2 participants