Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Issues 76 - Fix changes to wikipedia skill so that tests pass #77

Merged
merged 2 commits into from
Jun 1, 2016

Conversation

ethanaward
Copy link
Contributor

I changed the wikipedia skill so that it triggers on 'what does wikipedia say about'. I also removed 'who is' and 'who was' in the vocab, as those will be handled by Wolfram.

@ryanleesipes
Copy link

@ethanaward please coordinate with @jdorleans to see why the tests aren't working on this PR.

@ryanleesipes ryanleesipes added the Type: Enhancement - proposed New proposal for a feature that is not currently a priority on the roadmap. label Jun 1, 2016
@jdorleans
Copy link
Contributor

I thought I had commented on this PR a few days ago. Have you guys created another PR for this task?

@ryanleesipes
Copy link

Yeah, this is a different (clean) PR.

@jdorleans
Copy link
Contributor

Ok, the changes seems reasonable, but before merging I'd like to know if anyone has tested it.

@ethanaward
Copy link
Contributor Author

I tested it using ./start.sh unittest on my own, and the tests passed. @ryanleesipes was wondering why the tests weren't showing here, though.

@jdorleans
Copy link
Contributor

ok, this seems to be a forked branch which means it does not exist into our repo. As a result, Travis won't be able to find it and run the tests

@ryanleesipes ryanleesipes merged commit 4d6defd into MycroftAI:master Jun 1, 2016
@ethanaward ethanaward deleted the bugfix/issues-76 branch June 10, 2016 19:15
@waldyrious waldyrious mentioned this pull request Jan 31, 2018
NeonDaniel pushed a commit to NeonDaniel/mycroft-core that referenced this pull request Jun 8, 2022
* feat/fallback_stt

* add vosk

* license tests

* add stt unittests

* add stt unittests

* test default plugins

* add fallback stt unittests

* disable py3.10 unittests temporarily

build tests will still fail for py3.10, once they pass reenable py3.10 unittests

* test bad fallback stt configs

Co-authored-by: jarbasai <jarbasai@mailfence.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement - proposed New proposal for a feature that is not currently a priority on the roadmap.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants