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.

Sign up for GitHub

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

Jump to bottom

chore:upgrade some dependent packages to eliminate dependencies on some archived repositories #767

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

chore:upgrade some dependent packages to eliminate dependencies on some archived repositories #767

wants to merge 31 commits into from

Conversation

No-SilverBullet
Copy link
Contributor

What this PR does:
Upgrade some dependent packages to eliminate dependencies on some archived repositories as much as possible

包名 仓库地址 依赖类型 是否可以消除 解决办法/无法消除原因
github.com/benbjohnson/clock https://github.com/benbjohnson/clock 间接依赖 是 升级go.uber.org/zap到最新版本v1.27.0
github.com/mitchellh/copystructure https://github.com/mitchellh/copystructure 间接依赖 否 升级koanf到最新版本（v2.1.2）后仍然存在
github.com/mitchellh/mapstructure https://github.com/mitchellh/mapstructure 间接依赖 是 升级koanf到最新版本（v2.1.2）
github.com/mitchellh/reflectwalk https://github.com/mitchellh/reflectwalk 间接依赖 否 升级koanf到最新版本（v2.1.2）后仍然存在
github.com/pkg/errors https://github.com/pkg/errors 直接依赖 否 errors包稳定且广泛使用，没有必要移除
github.com/golang/mock https://github.com/golang/mock 直接依赖 否 go官方包，稳定且广泛使用，没有必要移除

Which issue(s) this PR fixes:
#755

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

No-SilverBullet and others added 29 commits November 10, 2024 13:06
@No-SilverBullet
feat:add more linter
d84f3c0
@No-SilverBullet
Merge pull request #1 from No-SilverBullet/feat/add_linter
aaae50c 
feat:add more linter
@No-SilverBullet
feat:change golangclilint version to 1.57.x to support more linter
0fc667d
@No-SilverBullet
Merge pull request #2 from No-SilverBullet/feat/add_linter
6a4bebe 
feat:change golangclilint version to 1.57.x to support more linter
@No-SilverBullet
feat:adjust lint conf and adjust the code to pass the check
358f4ac
@No-SilverBullet
Merge pull request #3 from No-SilverBullet/feat/add_linter
ac85f94 
feat:adjust lint conf and adjust the code to pass the check
@luky116
Merge branch 'master' into master
5c23b6c
@luky116
Merge branch 'master' into master
fd25729
@luky116
Merge branch 'master' into master
202cd26
@No-SilverBullet
Merge branch 'apache:master' into master
eb16366
@No-SilverBullet
style: format some code; fix: some sql statement or rows was not been…
f20acfe 
… closed
@No-SilverBullet
feat:close session when send heart beat message failed
ce334f7
@No-SilverBullet
Merge pull request #4 from No-SilverBullet/feat/check_session_alive
758a214 
feat:close session when send heart beat message failed
@No-SilverBullet
Merge pull request #5 from No-SilverBullet/refactor/format
5add578 
style: format some code; fix: some sql statement or rows was not been…
@No-SilverBullet
Merge branch 'apache:master' into master
00bce0b
@solisamicus
refactor: change GettyRemoting from singleton to GettyRemotingClient …
5f2e225 
…member
@marsevilspirit
remove rpc client
10bf844
@marsevilspirit
Merge branch 'master' into refactor/getty
b92d5bf
@solisamicus
refactor: complete GettyRemoting refactoring
a4a789f
@solisamicus
refactor: change getter name, improve variable naming and remove unus…
265e45e 
…ed fields
@marsevilspirit
update something
b54d4b7
@No-SilverBullet
Merge pull request #9 from marsevilspirit/refactor/getty
f5f72b6 
update session manager
@No-SilverBullet
feat:add heart-beat failed retry times
a4caadb
@No-SilverBullet
Merge pull request #8 from No-SilverBullet/refactor/getty
dd541f8 
refactor/getty
@No-SilverBullet
Refactor getty.
001e49c 
>
>
Co-authored-by: marsevilspirit <[email protected]>
Co-authored-by: solisamicus <[email protected]>
Co-authored-by: No-SilverBullet  <[email protected]>
@No-SilverBullet @marsevilspirit @solisamicus
Refactor getty.
51d1cae 
Co-authored-by: marsevilspirit <[email protected]>
Co-authored-by: solisamicus <[email protected]>
Co-authored-by: No-SilverBullet  <[email protected]>
@No-SilverBullet
Merge branch 'apache:master' into master
fa341cb
@No-SilverBullet
upgrade some dependent packages
2bce5fe
@No-SilverBullet
Merge pull request #10 from No-SilverBullet/chore/clean-go-mod
afc48d9 
upgrade some dependent packages
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

slievrly
slievrly reviewed Jan 24, 2025
View reviewed changes
Copy link
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slievrly slievrly slievrly left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
coding
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@No-SilverBullet @slievrly @luky116 @solisamicus @marsevilspirit