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

remove getSpeechForSpelling alias for getSpellingSpeech #12145

merged 4 commits into from Mar 11, 2021


Copy link

@seanbudd seanbudd commented Mar 10, 2021

Link to issue number:

Part of #12123

Implements deprecation announced in: #10593

Summary of the issue:

speech.getSpeechForSpelling was deprecated a year ago

Description of how this pull request fixes the issue:

speech.getSpeechForSpelling is removed in favour of the equivalent alias speech.getSpellingSpeech

Testing strategy:

Search the code base for usages of getSpeechForSpelling and confirm there is none

find . -type f -name '*.py' -exec grep -P "\bgetSpeechForSpelling\b" {} +

General user testing in the alpha/beta stages will confirm lack of changes

Known issues with pull request:


Change log entry:

Developer Changes

-`speech.getSpeechForSpelling` has been removed - use `speech.getSpellingSpeech` (#12145)

Code Review Checklist:

  • Pull Request description is up to date.
  • Unit tests.
  • System (end to end) tests.
  • Manual tests.
  • User Documentation.
  • Change log entry.
  • Context sensitive help for GUI changes.

@seanbudd seanbudd mentioned this pull request Mar 10, 2021
18 tasks
@seanbudd seanbudd self-assigned this Mar 10, 2021
@seanbudd seanbudd added the deprecated/2021.1 Label used to track deprecations due for removal in the 2021.1 release label Mar 10, 2021
@seanbudd seanbudd added this to the 2021.1 milestone Mar 10, 2021
Copy link

See test results for failed build of commit 84f3453c0b

@seanbudd seanbudd merged commit 8e4b189 into master Mar 11, 2021
@seanbudd seanbudd deleted the dep-10593 branch March 11, 2021 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
deprecated/2021.1 Label used to track deprecations due for removal in the 2021.1 release
None yet

Successfully merging this pull request may close these issues.

None yet

3 participants