Skip to content
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.

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

Install cli error on Deno 1.12 #5

Closed
KyleJune opened this issue Aug 7, 2021 · 2 comments · Fixed by #7
Closed

Install cli error on Deno 1.12 #5

KyleJune opened this issue Aug 7, 2021 · 2 comments · Fixed by #7

Comments

@KyleJune
Copy link

KyleJune commented Aug 7, 2021

kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ deno --version
deno 1.12.2 (release, x86_64-unknown-linux-gnu)
v8 9.2.230.14
typescript 4.3.5
kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ deno install --allow-read=. --allow-write=. --allow-net=deno.land,esm.sh,cdn.esm.sh -fq https://deno.land/x/twd@v0.4.5/cli.ts
error: TS2345 [ERROR]: Argument of type '(path: string | URL, options?: ReadFileOptions | undefined) => Promise<string>' is not assignable to parameter of type '(value: string, index: number, array: string[]) => Promise<string>'.
  Types of parameters 'options' and 'index' are incompatible.
    Type 'number' is not assignable to type 'ReadFileOptions | undefined'.
    await Promise.all(files.map(Deno.readTextFile)),
                                ~~~~~~~~~~~~~~~~~
    at https://deno.land/x/twd@v0.4.5/cli.ts:163:33
kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ 

I was able to get it to work by downgrading deno to 1.11.5

kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ deno --versiondeno 1.11.5 (release, x86_64-unknown-linux-gnu)
v8 9.1.269.35
typescript 4.3.2
kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ deno install --allow-read=. --allow-write=. --allow-net=deno.land,esm.sh,cdn.esm.sh -fq https://deno.land/x/twd@v0.4.5/cli.ts
✅ Successfully installed twd
/home/kyle/.deno/bin/twd
kyle@kyle-XPS-15-9560:~/Projects/deno/tailwinds-example$ 
@kt3k
Copy link
Owner

kt3k commented Aug 9, 2021

@KyleJune Thank you for reporting! I'll look into it.

@kt3k
Copy link
Owner

kt3k commented Aug 10, 2021

Should be fixed with v0.4.7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants