Skip to content

Commit

Permalink
Fix spelling
Browse files Browse the repository at this point in the history
  • Loading branch information
jbampton committed Dec 10, 2020
1 parent 28e7712 commit b2d6ddf
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion docs/coho-release-process.md
Expand Up @@ -62,7 +62,7 @@ See if any dependencies are outdated

(cd cordova-coho && npm outdated --depth=0)

Update them in each project's `package.json` file. Make sure to run through the test section below for compatability issues. The `--depth=0` prevents from listing dependencies of dependencies.
Update them in each project's `package.json` file. Make sure to run through the test section below for compatibility issues. The `--depth=0` prevents from listing dependencies of dependencies.

## Update Release Notes & Version

Expand Down
4 changes: 2 additions & 2 deletions docs/processing-pull-requests.md
Expand Up @@ -75,7 +75,7 @@ _Thanks for the pull request. I've had a look at it and think it looks good. Bef

There are two ways to merge a PR.

The goal is to have one or more commits in `master` that are connected to both the original author and you as the commiter, and include the JIRA issue number in the form CB-#### (no "[]"'s, no ":").
The goal is to have one or more commits in `master` that are connected to both the original author and you as the committer, and include the JIRA issue number in the form CB-#### (no "[]"'s, no ":").

Example commit message:

Expand All @@ -99,7 +99,7 @@ This command will do the following:
* Attempt a `--ff-only` merge to master. If this fails, then:
* Perform a rebase of the `pr/pr#` branch.
* Attempt a `--ff-only` merge to master.
* On success, it will modify the last commit's message to include. 'This closes #pr' to ensure the corresponding PR closes on pushing to remote.
* On success, it will modify the last commits message to include. 'This closes #pr' to ensure the corresponding PR closes on pushing to remote.

You then should:
* Squash as many commits as is reasonable together.
Expand Down
2 changes: 1 addition & 1 deletion spec/gitutil.spec.js
Expand Up @@ -45,7 +45,7 @@ describe('gitutil unit tests', function () {
expect(executil.ARGS.calls.argsFor(0)[0]).toEqual('git status --porcelain');
}, TIMEOUT);

it('Test#004 : reseting from origin', function * () {
it('Test#004 : resetting from origin', function * () {
yield gitutilJS.resetFromOrigin();
expect(executil.execHelper.calls.count()).toEqual(1);
expect(executil.ARGS.calls.count()).toEqual(1);
Expand Down
2 changes: 1 addition & 1 deletion src/executil.js
Expand Up @@ -37,7 +37,7 @@ exports.verbose = false;
// silent == true or 1 ==> don't print command, don't print output
// silent == 2 ==> don't print command, print output
// silent == 3 ==> print command, don't print output
// TODO: this function should be consolidated to promises, and shouldnt take win/fail callbacks.
// TODO: this function should be consolidated to promises, and shouldn't take win/fail callbacks.
// some async confusion here
function execHelper (cmdAndArgs, silent, allowError, win, fail) {
// there are times where we want silent but not allowError.
Expand Down

0 comments on commit b2d6ddf

Please sign in to comment.