Skip to content

Commit

Permalink
minor doc changes
Browse files Browse the repository at this point in the history
  • Loading branch information
mikkelfj committed May 18, 2011
1 parent 23af5e0 commit 5000050
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
8 changes: 4 additions & 4 deletions doc/api/shell.md
Expand Up @@ -14,7 +14,7 @@ the shell name is not the site name but `this.issuer` which is a
longer unique id including the site name for better logging
consistency.

See also `sudo` operation below. This also apply when running shells
See also `sudo` operation below. This also applies when running shells
under job control.

### Standalone Shell Usage
Expand Down Expand Up @@ -55,9 +55,9 @@ to a single string joined by ' && ', like the last command below:
host.run("ls && touch hello2.test");


Site configurations are used by the `poly.jobs()`, partially to
initialise remote shells. Here is an example using just site
configurations and shells without job control.
Site configurations are used by `poly.jobs()`, partially to initialise
remote shells. Here is an example using just site configurations and
shells without job control:

var poly = require('poly');
var sites = poly.sites();
Expand Down
7 changes: 4 additions & 3 deletions doc/getting_started.md
Expand Up @@ -83,7 +83,7 @@ for more inspiration.
this.shell.run("mydeployscript.sh", function(ec, capture) {
status.ok = !ec;
if(ec)
this.site.adminAlert("deploy failed on " + site.name, capture());
this.site.adminAlert("deploy failed on " + site.name, capture.out());
cb(); // don't fail here, next job should check status
});

Expand Down Expand Up @@ -138,6 +138,7 @@ Roles are used to name groups of sites in a server cluster. This makes
it easy to assign jobs to specific sites, and also to configure
multiple sites consistently with common settings.

Shells are can redirect output, so it is possible to integrate the job
runner with a web application framework such as `Express` for `Node.js`.
Shells can redirect output, so it is possible to integrate the job
runner with a web application framework such as `Express` for
`Node.js`.

0 comments on commit 5000050

Please sign in to comment.