diff options
author | Christian Clauss <cclauss@me.com> | 2021-11-12 09:39:02 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-11-12 09:39:02 +0100 |
commit | 102ef25fdf27504f3ac7392ce8dbfe83485fb25a (patch) | |
tree | 8a105276486cb2c9e8e9ce65aa1b5e824e524c4b | |
parent | 2dab655765499e9b05b8f097ab446ddee8f9adca (diff) | |
download | ninja-102ef25fdf27504f3ac7392ce8dbfe83485fb25a.tar.gz |
Fix typos discovered by codespell (#2047)
% [`codespell --count --ignore-words-list=fo .`](https://pypi.org/project/codespell/)
```
./ninja/src/missing_deps.cc:119: dependecy ==> dependency
./ninja/src/includes_normalize-win32.cc:51: funcation ==> function
./ninja/src/eval_env.h:58: invokable ==> invocable
./ninja/src/missing_deps_test.cc:155: beacuse ==> because
./ninja/src/deps_log_test.cc:393: unparseable ==> unparsable
```
-rw-r--r-- | .github/workflows/linux.yml | 3 | ||||
-rw-r--r-- | src/deps_log_test.cc | 2 | ||||
-rw-r--r-- | src/eval_env.h | 2 | ||||
-rw-r--r-- | src/includes_normalize-win32.cc | 2 | ||||
-rw-r--r-- | src/missing_deps.cc | 2 | ||||
-rw-r--r-- | src/missing_deps_test.cc | 2 |
6 files changed, 8 insertions, 5 deletions
diff --git a/.github/workflows/linux.yml b/.github/workflows/linux.yml index 80c88c6..3c93e00 100644 --- a/.github/workflows/linux.yml +++ b/.github/workflows/linux.yml @@ -13,6 +13,9 @@ jobs: image: centos:7 steps: - uses: actions/checkout@v2 + - uses: codespell-project/actions-codespell@master + with: + ignore_words_list: fo,wee - name: Install dependencies run: | curl -L -O https://github.com/Kitware/CMake/releases/download/v3.16.4/cmake-3.16.4-Linux-x86_64.sh diff --git a/src/deps_log_test.cc b/src/deps_log_test.cc index 1c29d89..13fcc78 100644 --- a/src/deps_log_test.cc +++ b/src/deps_log_test.cc @@ -390,7 +390,7 @@ TEST_F(DepsLogTest, Truncated) { DepsLog log; EXPECT_TRUE(log.Load(kTestFilename, &state, &err)); if (!err.empty()) { - // At some point the log will be so short as to be unparseable. + // At some point the log will be so short as to be unparsable. break; } diff --git a/src/eval_env.h b/src/eval_env.h index ca7daa4..677dc21 100644 --- a/src/eval_env.h +++ b/src/eval_env.h @@ -55,7 +55,7 @@ private: TokenList parsed_; }; -/// An invokable build command and associated metadata (description, etc.). +/// An invocable build command and associated metadata (description, etc.). struct Rule { explicit Rule(const std::string& name) : name_(name) {} diff --git a/src/includes_normalize-win32.cc b/src/includes_normalize-win32.cc index 5d52943..081e364 100644 --- a/src/includes_normalize-win32.cc +++ b/src/includes_normalize-win32.cc @@ -48,7 +48,7 @@ bool IsPathSeparator(char c) { } // Return true if paths a and b are on the same windows drive. -// Return false if this funcation cannot check +// Return false if this function cannot check // whether or not on the same windows drive. bool SameDriveFast(StringPiece a, StringPiece b) { if (a.size() < 3 || b.size() < 3) { diff --git a/src/missing_deps.cc b/src/missing_deps.cc index 78feb49..de76620 100644 --- a/src/missing_deps.cc +++ b/src/missing_deps.cc @@ -116,7 +116,7 @@ void MissingDependencyScanner::ProcessNodeDeps(Node* node, Node** dep_nodes, // rebuild this target when the build is reconfigured", but build.ninja is // often generated by a configuration tool like cmake or gn. The rest of // the build "implicitly" depends on the entire build being reconfigured, - // so a missing dep path to build.ninja is not an actual missing dependecy + // so a missing dep path to build.ninja is not an actual missing dependency // problem. if (deplog_node->path() == "build.ninja") return; diff --git a/src/missing_deps_test.cc b/src/missing_deps_test.cc index 7b62e6c..db66885 100644 --- a/src/missing_deps_test.cc +++ b/src/missing_deps_test.cc @@ -152,7 +152,7 @@ TEST_F(MissingDependencyScannerTest, CycleInGraph) { CreateInitialState(); CreateGraphDependencyBetween("compiled_object", "generated_header"); CreateGraphDependencyBetween("generated_header", "compiled_object"); - // The missing-deps tool doesn't deal with cycles in the graph, beacuse + // The missing-deps tool doesn't deal with cycles in the graph, because // there will be an error loading the graph before we get to the tool. // This test is to illustrate that. std::string err; |