Merge pull request #291 from tkhshtsh0917/fix/chinese-normalization-p…

…inyin_feature_test_failed fix: fixed `chinese-normalization-pinyin` feature test failed
meilisearch · May 30, 2024 · dc29b7c · dc29b7c
2 parents 9234a07 + 7aa4d14
commit dc29b7c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -27,7 +27,7 @@ jobs:
     - name: Run tests with japanese-transliteration on
       run: cargo test --verbose --features japanese-transliteration
     - name: Run tests with chinese-normalization-pinyin on
-      run: cargo test --verbose --features chinese chinese-normalization-pinyin
+      run: cargo test --verbose --features chinese-normalization-pinyin
     - name: Run tests with swedish-recomposition on
       run: cargo test --verbose --features swedish-recomposition
     - name: Run irg-kvariants tests

diff --git a/charabia/src/normalizer/chinese.rs b/charabia/src/normalizer/chinese.rs
@@ -31,7 +31,7 @@ impl CharNormalizer for ChineseNormalizer {
 
                 with_tone.to_string()
             }
-            None => kvariant, // e.g. 杤
+            None => kvariant.to_string(), // e.g. 杤
         };
 
         Some(kvariant.into())