Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(filter): url fragment deduplication #101

Merged
merged 5 commits into from
Jun 26, 2022
Merged

Conversation

PIGfaces
Copy link
Contributor

@PIGfaces PIGfaces commented Jun 8, 2022

改动说明

本次改动涉及文件比较多,包含代码风格调整、添加一些帮助文件、去重功能添加针对 fragment 的去重

代码风格

  • 代码解耦:分离了一些业务代码和逻辑代码
  • 文件重命名
  • package name 重命名,便于其他仓库能直接在代码中 import 而不用 fork 之后重命名

Bug

其他

  • 添加 makefile 解放双手
  • 支持 darwin、linux 平台 amd64 和 arm 版本的编译

@Qianlitp
Copy link
Owner

Qianlitp commented Jun 8, 2022

nice,都是一些很有用的改动。
涉及的文件比较多,我需要一些时间review下代码

@Qianlitp Qianlitp merged commit dbf7064 into Qianlitp:master Jun 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants