# aragonSDK standup 4 notes

Previous standup nodes: https://hackmd.io/@ajsantander/Syezb8dcr

## Alejandro
* Related issue: https://github.com/aragon/aragon-cli/issues/858
* Proposed to move standup time 1 hour later: accepted.
* Proposed that we all start adding reviewers in https://github.com/aragon/aragon-cli/issues/858 so we don’t have stalled PRs. To do so, add yourself in the issue description on the corresponding line.
* Suggests having functions as pure as possible.
* Will be working on refactoring apm commands

## Daniel
* Related issue: https://github.com/aragon/aragon-cli/issues/866
* Related issue: https://github.com/aragon/aragon-cli/issues/858
* Last week, worked on adding typescript to the repo.
* This week, will finish refactoring `ipfs` and `dao token` commands.
* Will also work on updating linting rules and git hooks
* About the reporter, it is mostly needed in the middlewares for error handling purposes. Will perhaps be moved at some point.

## Gabi
* Related issue: https://github.com/aragon/aragon-cli/issues/852
* Related issue: https://github.com/aragon/aragon-cli/issues/858
* Worked on the `State of the aragonCLI` document: https://docs.google.com/document/d/1eDY49pCRT3ttYXe8OdXUJgT9s1wjVwqRSCNf_TzXLxA/edit?usp=sharing
* This week, will put on hold the document and work on code refactoring.
* Survey has now 16 responses! 🎉

## Mathew
* Concerns about `listr`'s context object. Proposed we use it as less as possible or use variables.
* Proposed we consider migrating to another library after the refactoring
* Anyone having suggestions should add them to the **Aragon SDK document**: https://docs.google.com/document/d/14wqx_hEif6M_8g1fqNctdsVSbjVN7InYblr04wZ7CyQ/edit
* Last week, finished merging the last pull requests before refactoring.
* Also worked on fixing e2e-tests on master
* Will work on `dao exec` command this week

## Lion
* Questions about the current structure of the CLI:
* Why are ABIs imported inline instead of at the top of the file? *Answer*: To fetch their content “on-demand”.
* Why the apm parameter contains an ipfs config object? *Answer*: At first, only the apm was using ipfs. It probably should not be nested anymore.
* Why use ava for tests? *Answer*: Consistency with other Aragon projects.
* Proposed an example of typed variables with jsdoc: https://github.com/dapplion/aragon-cli/blob/d9ab8515fb7ae9c64332d3872abc3daef040c6d1/packages/aragon-cli/src/commands/dao_cmds/acl_cmds/view.js#L49