Skip to content

Commit ac6982d

Browse files
authored
Comma after the introductory phrase, in the three places missing one (#224)
Reported across #221, #222 and #223 by lunar-me, one comma each. Taken together here because each of those also carried two changes already on main, so three rebases would have produced three near-empty diffs.
1 parent 2dc6c2f commit ac6982d

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

‎docs/deployment.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ enable it for you.
3333
**Not in it:**
3434

3535
**The supervisor.** It gives each Bot its own container, which needs a Docker socket, which no
36-
serverless container platform permits. Without it every Bot shares the one browser, exactly as they
36+
serverless container platform permits. Without it, every Bot shares the one browser, exactly as they
3737
do on a laptop with no supervisor configured. A shared browser means shared logins, shared files and
3838
shared session between Bots, which is fine for a deployment where one team trusts its own Bots and
3939
is not fine as a boundary between tenants.
@@ -79,7 +79,7 @@ against a host that is not there. Set it, with `MANAGED_AGENT_TOKEN`, only when
7979
reachable from this container. Unset it if your `.env` still has the laptop default
8080
`http://localhost:4201/ag-ui`.
8181

82-
**Authentication is required.** With no identity provider configured the deployment refuses to start,
82+
**Authentication is required.** With no identity provider configured, the deployment refuses to start,
8383
because a public URL where every visitor is an administrator fails silently: it looks like it works.
8484
Configure Google, Microsoft or Okta, or set `OPENBOT_SINGLE_USER=true` to say you meant an open
8585
deployment. `NODE_ENV` does not affect this.
@@ -94,7 +94,7 @@ never missing on a laptop. The app does not depend on any of them, but sign-in c
9494
With `EMBEDDED_POSTGRES=on` they run at start and there is nothing to do. There is exactly one
9595
process and no deploy pipeline, so the alternative would be a runbook.
9696

97-
With an external database they are a release step, not a start step. Two replicas starting together
97+
With an external database, they are a release step, not a start step. Two replicas starting together
9898
would race, and a failed migration should stop a deploy rather than leave a half-migrated database
9999
serving traffic.
100100

0 commit comments

Comments
 (0)