New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

283 explicit error on missing xlsx field #284

Merged

lrromero merged 2 commits into master from 283-explicit-error-on-missing-xlsx-field

Oct 4, 2019

Contributor

lrromero commented Oct 3, 2019

Closes #283

Atrapa los key error de la lectura de catalogos xlsx y lo levanta como una excepción NonParseableCatalog
Agrega un caso de test donde ocurre

lrromero added 2 commits

October 3, 2019 17:37


          Atrapo KeyErrors en los catalogos xlsx con campos faltantes

fd2448f


          Tests para catalogo xlsx con field faltante

d14dba0

coveralls commented Oct 3, 2019 •

edited

Coverage increased (+0.1%) to 83.685% when pulling d14dba0 on 283-explicit-error-on-missing-xlsx-field into c284204 on master.

lucaslavandeira approved these changes

View reviewed changes

lrromero merged commit 879da46 into master

lrromero deleted the 283-explicit-error-on-missing-xlsx-field branch

October 4, 2019 14:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment