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

Add some more plugin docs #6618

Merged
merged 2 commits into from Apr 5, 2019

Conversation

@msullivan
Copy link
Collaborator

commented Apr 3, 2019

  • Link to plugin changes announcement issue (#6617)
  • Document get_additional_deps()
Add some more plugin docs
 * Link to plugin changes announcement issue (#6617)
 * Document get_additional_deps()

@msullivan msullivan requested a review from ilevkivskyi Apr 3, 2019

@ilevkivskyi
Copy link
Collaborator

left a comment

Thank you! I have one wording suggestion (which also clarifies our policy).

no guarantees about backwards compatibility.

Backwards incompatible changes may be made without a deprecation period,
but we will attempt to announce them in

This comment has been minimized.

Copy link
@ilevkivskyi

ilevkivskyi Apr 3, 2019

Collaborator

I think we should just say "we will announce" (and actually do this :-))

This comment has been minimized.

Copy link
@msullivan

msullivan Apr 5, 2019

Author Collaborator

"WP:WEASEL" :P

@msullivan msullivan merged commit 83f75fe into master Apr 5, 2019

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@msullivan msullivan deleted the plugin-docs branch Apr 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.