refactor: replace golang.org/x/exp with stdlib#774
Conversation
WalkthroughThis pull request updates import statements in two Go files. In both Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
m-Peter
left a comment
There was a problem hiding this comment.
Awesome 🙌 Thanks for your contribution!
|
@StevenMia There are some linting errors, could you try to resolve them? |
It seems that the CI failure is not directly related to this modification. However, I'm happy to investigate the cause. I submitted an empty file to trigger the CI, but it requires approval from the maintainer. |
This is what I see: Error: File is not properly formatted (goimports)So most likely, the 2 files changed by this PR, have some formatting issue. |
Signed-off-by: StevenMia <flite@foxmail.com>
Oh, yes, I have modified this problem. Please approve CI to rerun it. |
|
Currently on my own branch, it has been passed. |
|
Great! They all passed! |
Closes: #???
Description
Since Go 1.21, the functions in x/exp used here can already be replaced by functions from the standard library
For contributor use:
masterbranchFiles changedin the Github PR explorerSummary by CodeRabbit