search

MycroftAI / documentation

Mycroft.AI documentation for all public facing technical components.
https://mycroft.ai/documentation
Apache License 2.0
80 stars 90 forks source link

Remove references to code that's been removed #121

Closed strugee closed 5 years ago

strugee commented 5 years ago

See https://github.com/MycroftAI/mycroft-skills/commit/da6066e97e019f281d5096f33508ea8ee3208929#diff-bbacc29aecae5c3e37042cf23b14652e

strugee commented 5 years ago

Actually, looks like this whole document might need to be reworked. The section about converting to use decorators is outdated too, since as of that commit the template already uses decorators.

forslund commented 5 years ago

You're quite right. The entire document needs to be updated. If you like to tackle that as well we can hold off on merging this. Otherwise I think this can be merged when the CLA lands and we can continue the work in separate PR's.

strugee commented 5 years ago

Honestly I'm just working through the introductory materials myself so I'm not sure how much I'll really be able to explain :P

Why don't you go ahead and merge this when I'm done with the CLA process and I'll file a followup issue to clean the rest up? Then if/when I feel like I've got enough experience I can go ahead and do it myself, or if someone gets to it before me they can take it and run with it too.