surgio icon indicating copy to clipboard operation
surgio copied to clipboard

feat: 优化关键词排序算法

Open baranwang opened this issue 3 years ago • 12 comments

  • 增加排序评分
  • 支持分组关键词排序:eg:
{
// ...
  customFilters: {
    fallback: utils.useSortedKeywords([
      ['HC', 'AIA', 'IEPL'],
      ['香港', '台湾', '新加坡', '日本'],
    ]), // 香港 HC, 台湾 HC, 日本 HC, 香港 AIA, 台湾 AIA, 新加坡 AIA, 日本 AIA ....
  },
// ...
}

baranwang avatar Dec 01 '21 06:12 baranwang

✔️ Deploy Preview for surgio-documentation ready!

🔨 Explore the source changes: 4991c2cf2fa8bd3ae34e34073091d518b6c4e999

🔍 Inspect the deploy log: https://app.netlify.com/sites/surgio-documentation/deploys/61b4791d5b5fbf0007685605

😎 Browse the preview: https://deploy-preview-163--surgio-documentation.netlify.app

netlify[bot] avatar Dec 01 '21 06:12 netlify[bot]

ping @geekdada

baranwang avatar Dec 09 '21 03:12 baranwang

感谢贡献代码!

稍后有空时 review。

geekdada avatar Dec 09 '21 09:12 geekdada

请问有没有空更新一下文档?

geekdada avatar Dec 10 '21 22:12 geekdada

请检查一下测试用例,没有通过

geekdada avatar Dec 11 '21 11:12 geekdada

请检查一下测试用例,没有通过

我看了一下,测试用例master分支也是跑不通的… 好像不是这个pr的问题哎~

baranwang avatar Dec 11 '21 11:12 baranwang

覆盖率那个可以暂时忽略,其他的是可以通过的。

geekdada avatar Dec 11 '21 14:12 geekdada

image 哦?我这边测试master和我pr的分支报错是一样的

baranwang avatar Dec 11 '21 15:12 baranwang

我在本地和 CI 都跑了 master 分支,是没问题的。

geekdada avatar Dec 11 '21 16:12 geekdada

请问怎么样了呢

geekdada avatar Dec 17 '21 19:12 geekdada

master上单元测试同样过不了啊,没有找到问题的点🙃

baranwang avatar Dec 20 '21 03:12 baranwang

等我今天checkout你的分支看一下。

geekdada avatar Dec 20 '21 08:12 geekdada