Should the default `short_errors` option really ignore errors?

[tvogel]

... and not only abbreviate the display?

In my opinion, it should only abbreviate the display because it is not great if executing a notebook just skips SQL errors. To achieve this, this line only needed to be unindented by one level:

ipython-sql/src/sql/magic.py

Line 261 in eb27484

raise e

Is it maybe unintended indentation anyway?