Skip to content

Conversation

@dario-piotrowicz
Copy link
Member

@dario-piotrowicz dario-piotrowicz commented Jan 1, 2025

Summary

Add info regarding the newly introduced --strict-vars option for the wrangler types command

Documentation checklist

Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 files reviewed, 1 total issue(s) found.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 1, 2025

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: ddd7c40
Status: ✅  Deploy successful!
Preview URL: https://f5178905.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://dario-wrangler-types-loose-e.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

github-actions bot commented Jan 1, 2025

@dario-piotrowicz dario-piotrowicz changed the title add --loose-vars option to wrangler types command add --strict-vars option to wrangler types command Jan 2, 2025
@dario-piotrowicz dario-piotrowicz changed the title add --strict-vars option to wrangler types command add --strict-vars option to the wrangler types command Jan 2, 2025
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 files reviewed, 1 total issue(s) found.


Note: We resolved prior Hyperlint review comments because:

We updated our inline suggestion AI.

We do this to avoid keeping outdated or irrelevant comments around. We'll leave a new review with current comments below.

rename `--loose-vars` to `--strict-vars`
@dario-piotrowicz dario-piotrowicz force-pushed the dario/wrangler-types/loose-envs branch from bd47250 to 60ddd7e Compare January 2, 2025 13:32
@dario-piotrowicz dario-piotrowicz marked this pull request as ready for review January 21, 2025 13:45
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 files reviewed, 1 total issue(s) found.

@cloudflare cloudflare deleted a comment from hyperlint-ai bot Jan 21, 2025
@dario-piotrowicz dario-piotrowicz enabled auto-merge (squash) January 21, 2025 15:44
@dario-piotrowicz dario-piotrowicz enabled auto-merge (squash) January 21, 2025 16:01
@dario-piotrowicz dario-piotrowicz merged commit b71a6a9 into production Jan 21, 2025
11 of 12 checks passed
@dario-piotrowicz dario-piotrowicz deleted the dario/wrangler-types/loose-envs branch January 21, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants