Skip to content

Conversation

@dgw
Copy link
Member

@dgw dgw commented Jul 4, 2021

Description

Tin. But for completeness, a list:

  • msg() was replaced by say() a LONG time ago
  • the register()/unregister() methods are semi-internal plugin stuff, and were replaced by other methods in 7.1

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

* `msg()` was replaced by `say()` a LONG time ago
* the `register()`/`unregister()` methods (semi-internal plugin stuff)
@dgw dgw added Low Priority Breaking Change Stuff that probably should be mentioned in a migration guide Housekeeping Code cleanup, removal of deprecated stuff, etc. labels Jul 4, 2021
@dgw dgw added this to the 8.0.0 milestone Jul 4, 2021
@dgw dgw requested a review from a team July 4, 2021 23:23
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

YES

@dgw dgw merged commit b350357 into master Sep 26, 2021
@dgw dgw deleted the remove-deprecated-bot-methods branch September 26, 2021 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking Change Stuff that probably should be mentioned in a migration guide Housekeeping Code cleanup, removal of deprecated stuff, etc. Low Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants