workers-sdk icon indicating copy to clipboard operation
workers-sdk copied to clipboard

Strip down wrangler.toml templates

Open vicb opened this issue 1 year ago • 9 comments

What this PR solves / how to test

The wrangler.toml templates have 100+ lines of commented out template config.

I think it doesn't help users and potentially could make them think workers are more complex than they really are.

This is only a draft PR to get feedback.

What would be great is first to agree on the wording.

Then we should probably automatically inject the comments in all wrangler.toml instead of having to maintain 20 versions of those comments.

@irvinebroque @petebacondarwin thoughts?

Fixes N/A

Author has addressed the following

  • Tests
    • [ ] TODO (before merge)
    • [ ] Tests included
    • [ ] Tests not necessary because:
  • E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
    • [ ] I don't know
    • [ ] Required
    • [ ] Not required because:
  • Changeset (Changeset guidelines)
    • [ ] TODO (before merge)
    • [ ] Changeset included
    • [ ] Changeset not necessary because:
  • Public documentation
    • [ ] TODO (before merge)
    • [ ] Cloudflare docs PR(s):
    • [ ] Documentation not necessary because:

vicb avatar Oct 08 '24 08:10 vicb