Skip to content

feat: change say() method response type from void to Message (#1866) #5

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

Merged
merged 1 commit into from
Nov 1, 2019

Conversation

su-chang
Copy link
Owner

@su-chang su-chang commented Nov 1, 2019

  • feat: change say() method response type from void to Message

  • modify: bump wechaty and puppet version

  • Bump wechaty and wechaty-puppet version

  • 0.29.3

  • modify: check void response type, remove useless await

  • modify: check response type and refactory code style

  • modify: check msgId more strictly.

  • modify: update readme and JS doc, remove useless code.

  • modify: grammar fix

I'm submitting a...

[ ] Bug Fix
[ ] Feature
[ ] Other (Refactoring, Added tests, Documentation, ...)

Checklist

  • Commit Messages follow the Conventional Commits pattern
    • A feature commit message is prefixed "feat:"
    • A bugfix commit message is prefixed "fix:"
  • Tests for the changes have been added

Description

please describe the changes that you are making

for features, please describe how to use the new feature

please include a reference to an existing issue, if applicable

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

* feat: change say() method response type from void to Message

* modify: bump wechaty and puppet version

* Bump wechaty and wechaty-puppet version

* 0.29.3

* modify: check void response type, remove useless await

* modify: check response type and refactory code style

* modify: check msgId more strictly.

* modify: update readme and JS doc, remove useless code.

* modify: grammar fix
@su-chang su-chang merged commit 58fc1e1 into su-chang:master Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant