Skip to content

Commit

Permalink
refactor: excel to json list (DEV-431) (#155)
Browse files Browse the repository at this point in the history
* Don't open the file each time in the recursive function get_values_from_excel()

* instead of side effects, all methods should return the result they computed

* - make code more understandable by improving the logic
 - improve docstrings and annotations

* improve docstrings and annotations

* - various style/annotation improvements
 - remove side-effect of get_values_from_excel by making a copy of parentnode before returning it

* - correct mistake in logic
 - improve error message output

* - reconstruct the side-effect from before

* strip strings

* adapt 'expand_all_lists.py' to the refactorings

* elminate little bug

* introduce the side effect again

* rename variable

Co-authored-by: irinaschubert <irina.schubert@dasch.swiss>

* correct previous error

* improvements suggested by reviewer

Co-authored-by: irinaschubert <irina.schubert@dasch.swiss>

* implement reviewer's feedback

Co-authored-by: irinaschubert <irina.schubert@dasch.swiss>
  • Loading branch information
jnussbaum and irinaschubert committed Feb 17, 2022
1 parent be5b69f commit 8a8c9d0
Show file tree
Hide file tree
Showing 2 changed files with 160 additions and 129 deletions.

0 comments on commit 8a8c9d0

Please sign in to comment.