Fix apps lookup #119

2019-04-25T14:47:46Z

bumi commented

2019-04-25 14:47:46 +00:00

(Migrated from github.com)

We use the directories in ./apps to load available app names.
For this we need to make sure that we only use directories.

Some recent change broke this by adding a file in the apps directory.
So without this fix no DAO can be deployed.

We use the directories in ./apps to load available app names. For this we need to make sure that we only use directories. Some recent change broke this by adding a file in the apps directory. So without this fix no DAO can be deployed.

raucao (Migrated from github.com) reviewed 2019-04-25 14:59:04 +00:00

scripts/deploy-kit.js

raucao (Migrated from github.com) commented

2019-04-25 14:59:04 +00:00

  const apps = fs.readdirSync('./apps', { withFileTypes: true })
                 .filter(e => e.isDirectory())
                 .map(e => e.name);

```suggestion const apps = fs.readdirSync('./apps', { withFileTypes: true }) .filter(e => e.isDirectory()) .map(e => e.name); ```

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kredits/contracts#119