Conversation
There was a problem hiding this comment.
Para mim, parece estar tudo ok.
O TravisCI já executa os testes automaticamente ao abrir um PR, e segundo ele, não tivemos problemas.
Como eu disse anteriormente, o mais complicado é a validação dos estilos, já que hoje, os nossos testes unitários validam apenas o conteúdo gerado. Seria interessante adicionar mais testes no futuro que fazem algum tipo de cobertura em relação a estilos.
| var cell = createCell(wb, sheetRow, cellIndex, style, createHelper, valueClass); | ||
| setTypedValue(cell, value, valueClass); | ||
|
|
||
| var formula = (columnMD && columnMD.formula) ? columnMD.formula : null; |
There was a problem hiding this comment.
Aqui poderiamos usar apenas uma expressão booleana simples, já que o campo formula do metadado é só um boolean.
var formula = Boolean(columnMD && columnMD.formula);
Brunomachadob
left a comment
Brunomachadob
left a comment
There was a problem hiding this comment.
Esqueci de comentar anteriormente, mas como disse pessoalmente, seria legal conversarmos com o pessoal do Solução Certa, para realizar um teste com a planilha deles também. Acho que eles são os principais clientes desta lib.
2 participants
Bruno dá uma olhada nas funcionalidades que fiz. Se for interessante aceita o fork pra adicionar as novas features ao bitcode.
É bom também rodar os testes para garantir que nada foi quebrado.