Replace Keyword.get() with Keyword.fetch!() in execute/5, referring to migration source
darrenklein opened this issue · comments
Darren commented
In line 255, we see the use of Keyword.get()
to retrieve the name of the migration source table (and provide a default value if it's not in the config) -
however, in the init/1
function, we've already guaranteed that the default value will be provided if one hasn't been otherwise, so this logic is redundant. Since we can be sure that a value will have been set, we can use Keyword.fetch!()
instead.