Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

raise exception if given wrong slot name #1644

Closed
wants to merge 2 commits into from
Closed

Conversation

imLew
Copy link
Contributor

@imLew imLew commented Jan 28, 2019

Proposed changes:

  • fixes issue 1348

The problem was that if provided an incorrect name for a slot rasa would try to import it as a custom slot type
which failed by returning None. This None is now caught and an exception is thrown instead.

Status (please check what you already did):

  • made PR ready for code review
  • adapted some tests for the functionality

@imLew imLew requested review from akelad and removed request for akelad January 28, 2019 12:30
@codeclimate
Copy link

codeclimate bot commented Jan 28, 2019

Code Climate has analyzed commit 48b4c2f and detected 0 issues on this pull request.

View more on Code Climate.

@imLew imLew closed this Jan 28, 2019
@imLew imLew deleted the slotname_error branch January 28, 2019 14:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant