Add a spell check ci that will fail future PRs

.codespell/.codespellrc

@@ -0,0 +1,4 @@
+[codespell]
+quiet-level = 2
+skip = ./tests/gocase/util/slot.go
+ignore-words = ./.codespell/wordlist.txt

.codespell/requirements.txt

@@ -0,0 +1 @@
+codespell==2.2.5

.codespell/wordlist.txt

@@ -0,0 +1,3 @@
+exat
+smove
+ue

.github/workflows/spell-check.yml

@@ -0,0 +1,50 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+# A CI action that using codespell to check spell.
+# .codespell/.codespellrc is a config file.
+# .codespell/wordlist.txt is a list of words that will ignore word checks.
+# More details please check the following link:
+# https://github.com/codespell-project/codespell
+
+name: Spellcheck
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  build:
+    name: Spellcheck
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v3
+
+      - name: pip cache
+        uses: actions/cache@v3
+        with:
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt') }}
+          restore-keys: ${{ runner.os }}-pip-
+
+      - name: Install prerequisites
+        run: sudo pip install -r ./.codespell/requirements.txt
+
+      - name: Spell check
+        run: codespell --config=./.codespell/.codespellrc

kvrocks.conf

@@ -802,7 +802,7 @@ rocksdb.write_options.sync no
 # If yes, writes will not first go to the write ahead log,
 # and the write may get lost after a crash.
 #
-# Deafult: no
+# Default: no
 rocksdb.write_options.disable_wal no
 
 # If enabled and we need to wait or sleep for the write request, fails

src/cluster/slot_import.cc

@@ -105,7 +105,7 @@ void SlotImport::StopForLinkError(int fd) {
   // 2. Source server's migration process end after destination server has finished replication.
   // 3. The migration link closed by source server, then this function will be call by OnEvent.
   // 4. ClearKeysOfSlot can clear data although server is a slave, because ClearKeysOfSlot
-  //    deletes data in rocksdb directly. Therefor, it is necessary to avoid clearing data gotten
+  //    deletes data in rocksdb directly. Therefore, it is necessary to avoid clearing data gotten
   //    from new master.
   if (!svr_->IsSlave()) {
     // Clean imported slot data

src/cluster/slot_migrate.cc

@@ -363,7 +363,7 @@ Status SlotMigrator::sendSnapshot() {
   }
 
   LOG(INFO) << "[migrate] Succeed to migrate slot snapshot, slot: " << slot << ", Migrated keys: " << migrated_key_cnt
-            << ", Expired keys: " << expired_key_cnt << ", Emtpy keys: " << empty_key_cnt;
+            << ", Expired keys: " << expired_key_cnt << ", Empty keys: " << empty_key_cnt;
 
   return Status::OK();
 }

src/commands/cmd_cluster.cc

@@ -56,7 +56,7 @@ class CommandCluster : public Commander {
     }
 
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     if (subcommand_ == "keyslot") {
@@ -216,7 +216,7 @@ class CommandClusterX : public Commander {
     }
 
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     bool need_persist_nodes_info = false;

src/commands/cmd_server.cc

@@ -82,7 +82,7 @@ class CommandNamespace : public Commander {
  public:
   Status Execute(Server *svr, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     Config *config = svr->GetConfig();
@@ -173,7 +173,7 @@ class CommandFlushAll : public Commander {
  public:
   Status Execute(Server *svr, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     if (svr->GetConfig()->cluster_enabled) {
@@ -221,7 +221,7 @@ class CommandConfig : public Commander {
  public:
   Status Execute(Server *svr, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     Config *config = svr->GetConfig();
@@ -547,7 +547,7 @@ class CommandShutdown : public Commander {
  public:
   Status Execute(Server *srv, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     if (!srv->IsStopped()) {
@@ -842,7 +842,7 @@ class CommandBGSave : public Commander {
  public:
   Status Execute(Server *svr, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     Status s = svr->AsyncBgSaveDB();
@@ -858,7 +858,7 @@ class CommandFlushBackup : public Commander {
  public:
   Status Execute(Server *svr, Connection *conn, std::string *output) override {
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     Status s = svr->AsyncPurgeOldBackups(0, 0);
@@ -917,7 +917,7 @@ class CommandSlaveOf : public Commander {
     }
 
     if (!conn->IsAdmin()) {
-      return {Status::RedisExecErr, errAdministorPermissionRequired};
+      return {Status::RedisExecErr, errAdministratorPermissionRequired};
     }
 
     if (host_.empty()) {

src/commands/error_constants.h

@@ -26,7 +26,8 @@ inline constexpr const char *errInvalidSyntax = "syntax error";
 inline constexpr const char *errInvalidExpireTime = "invalid expire time";
 inline constexpr const char *errWrongNumOfArguments = "wrong number of arguments";
 inline constexpr const char *errValueNotInteger = "value is not an integer or out of range";
-inline constexpr const char *errAdministorPermissionRequired = "administor permission required to perform the command";
+inline constexpr const char *errAdministratorPermissionRequired =
+    "administrator permission required to perform the command";
 inline constexpr const char *errValueMustBePositive = "value is out of range, must be positive";
 inline constexpr const char *errNoSuchKey = "no such key";
 inline constexpr const char *errUnbalancedStreamList =

src/types/geohash.cc

@@ -312,7 +312,7 @@ uint8_t GeoHashHelper::EstimateStepsByRadius(double range_meters, double lat) {
   }
   step -= 2; /* Make sure range is included in most of the base cases. */
 
-  /* Wider range torwards the poles... Note: it is possible to do better
+  /* Wider range towards the poles... Note: it is possible to do better
    * than this approximation by computing the distance between meridians
    * at this latitude, but this does the trick for now. */
   if (lat > 66 || lat < -66) {
@@ -328,7 +328,7 @@ uint8_t GeoHashHelper::EstimateStepsByRadius(double range_meters, double lat) {
 
 /* Return the bounding box of the search area centered at latitude,longitude
  * having a radius of radius_meter. bounds[0] - bounds[2] is the minimum
- * and maxium longitude, while bounds[1] - bounds[3] is the minimum and
+ * and maximum longitude, while bounds[1] - bounds[3] is the minimum and
  * maximum latitude. */
 int GeoHashHelper::BoundingBox(GeoShape *geo_shape) {
   if (!geo_shape->bounds) return 0;

src/types/redis_geo.cc

@@ -320,7 +320,7 @@ void Geo::scoresOfGeoHashBox(GeoHashBits hash, GeoHashFix52Bits *min, GeoHashFix
  * 'max', appending them into the array of GeoPoint structures 'gparray'.
  * The command returns the number of elements added to the array.
  *
- * Elements which are farest than 'radius' from the specified 'x' and 'y'
+ * Elements which are farther than 'radius' from the specified 'x' and 'y'
  * coordinates are not included.
  *
  * The ability of this function to append to an existing set of points is

tests/cppunit/types/hash_test.cc

@@ -303,7 +303,7 @@ TEST_F(RedisHashTest, HRangeByLex) {
   EXPECT_EQ("key1", result[0].field);
   EXPECT_EQ("key2", result[1].field);
 
-  // inf and revered
+  // inf and reversed
   spec.minex = false;
   spec.maxex = false;
   spec.min = "-";

tests/gocase/unit/multi/multi_test.go

@@ -37,7 +37,7 @@ func TestMulti(t *testing.T) {
 	rdb := srv.NewClient()
 	defer func() { require.NoError(t, rdb.Close()) }()
 
-	t.Run("MUTLI / EXEC basics", func(t *testing.T) {
+	t.Run("MULTI / EXEC basics", func(t *testing.T) {
 		require.NoError(t, rdb.Del(ctx, "mylist").Err())
 		require.NoError(t, rdb.RPush(ctx, "mylist", "a").Err())
 		require.NoError(t, rdb.RPush(ctx, "mylist", "b").Err())