You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would be happy to submit a PR to fix this if it is wanted.
The simplest fix is to list all the possible exceptions, but actually, I think it would be more useful to catch at least some of the possible exceptions and wrap them in ExcelReaderException. But this is probably a breaking change.
It would be useful for methods like ExcelReaderFactory.CreateReader to document the possible exceptions that can be thrown.
For example, we had a corrupted .xlsx file that threw a System.IO.InvalidDataException, which surprisingly does not derive from IOException.
The text was updated successfully, but these errors were encountered: