epicweb-dev / rocket-rental

Rent Rockets from rocking awesome people
Other
277 stars 34 forks source link

Add missing brackets to Fly configuration #19

Closed szepeviktor closed 1 year ago

szepeviktor commented 1 year ago

Seems like they were missing. @kentcdodds

kentcdodds commented 1 year ago

Thanks for this @szepeviktor. I don't understand TOML syntax, so you could be right. However, the Fly docs have it the same way I do: https://fly.io/docs/reference/configuration/#services-concurrency

Have you tested your change in any way?

szepeviktor commented 1 year ago

No.

Those docs may be incorrect. Let us see! https://github.com/superfly/docs/pull/783

szepeviktor commented 1 year ago

We are in TOML World :) Please see https://github.com/superfly/docs/pull/783#issuecomment-1572794983

kentcdodds commented 1 year ago

TOML is super confusing. 😆