Skip to content

Conversation

@WalshyDev
Copy link
Member

@WalshyDev WalshyDev commented Dec 23, 2024

Summary

assets = {} syntax is easy to get wrong in TOML land, we've seen this time and time again with things like kv_namespaces. We have standardised on [xxxx] & [[xxxx]] across our docs. Let's make sure assets is following the same pattern here too.

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@cloudflare-workers-and-pages
Copy link

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 85e257a
Status: ✅  Deploy successful!
Preview URL: https://6a3974cb.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://walshy-update-assets-syntax.cloudflare-docs-7ou.pages.dev

View logs

@WalshyDev WalshyDev merged commit 6eb5f7e into production Dec 23, 2024
13 checks passed
@WalshyDev WalshyDev deleted the walshy/update-assets-syntax branch December 23, 2024 15:28
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/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants