rasa icon indicating copy to clipboard operation
rasa copied to clipboard

Published 20 hours ago • verified publisher icon RasaHQ

Fix spelling errors

Open thiagohdaqw opened this issue 1 year ago • 2 comments

Proposed changes:

  • Fixed spelling erros in docs and code

Status (please check what you already did):

  • [ ] added some tests for the functionality
  • [x] updated the documentation
  • [x] updated the changelog (please check changelog for instructions)
  • [ ] reformat files using black (please check Readme for instructions)

thiagohdaqw avatar Aug 16 '22 02:08 thiagohdaqw

@ancalita, done! sry for the delay

thiagohdaqw avatar Sep 06 '22 01:09 thiagohdaqw

@thiagohdaqw in order to merge this PR, we need to fix the Code Quality fail (the other failing job is flaky, so we can re-run it until it passes. The linter is a bit temperamental, so even if you had not modified those specific lines, it will check for any irregularities in the file you made edits, let us know if you need any direction with solving these errors. Once solved, you can run make lint in your terminal to check if everything passes.

Error: rasa/core/channels/mattermost.py:121:1: D102 Missing docstring in public method
Error: rasa/core/channels/slack.py:297:1: D202 No blank lines allowed after function docstring

ancalita avatar Sep 09 '22 09:09 ancalita