ampproject / ampproject/cloudflare-amp-optimizer

Install fails with error message

Open
#23 4 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
23
Forks
4
PR merge metrics
No merged PRs in 30d

Description

This might be the expected behavior, but I thought I'd document this here:

```
Need to install the following packages:
@cloudflare/wrangler
Ok to proceed? (y) y
⬇️ Installing cargo-generate v0.5.0...
🔧 Creating project called `my-worker`...
✨ Done! New project created /Users/sbenz/Projects/my-worker
🕵️ You can find your zone_id in the right sidebar of a zone's overview tab at https://dash.cloudflare.com
🕵️ You can find your account_id in the right sidebar of your account's Workers page
🕵️ You will need to update the following fields in the created wrangler.toml file before continuing:
- account_id
Error: values must be emitted before tables
npm ERR! code 1
npm ERR! path /Users/sbenz/Projects
npm ERR! command failed
npm ERR! command sh -c wrangler "generate" "my-worker" "https://github.com/ampproject/cloudflare-optimizer"

npm ERR! A complete log of this run can be found in:
npm ERR! /Users/sbenz/.npm/_logs/2021-04-14T10_03_16_967Z-debug.log
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the logged `wrangler generate` command and inspect the generated `wrangler.toml` before the `values must be emitted before tables` error. Confirm whether the failure is in project generation or installation, and consider the issue done when the command completes successfully with a usable generated project.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
cli, tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.