Skip to content

Bump dependency #50

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
Merged
yoshidan merged 1 commit into from
Dec 21, 2024
Merged

Bump dependency #50

yoshidan merged 1 commit into from
Dec 21, 2024
+147 −153

Conversation

@yoshidan
Copy link
Contributor

@yoshidan yoshidan commented Dec 20, 2024

bump deps
21fad21 
Signed-off-by: Naohiro Yoshida <naohiro.yoshida@datachain.jp>
@yoshidan yoshidan requested review from 3100 and siburu December 20, 2024 05:46
siburu
siburu approved these changes Dec 20, 2024
View reviewed changes
Copy link
Contributor

@siburu siburu left a comment

Choose a reason for hiding this comment

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

LGTM!

@siburu
Copy link
Contributor

siburu commented Dec 20, 2024

FYI: you can use go-1.21 to build modules that have go 1.22.0 directives.
At the time, necessary go toolchain >= 1.22.0 will be downloaded automatically.

@yoshidan yoshidan merged commit f95c9eb into main Dec 21, 2024
2 checks passed
@yoshidan yoshidan deleted the feature/bump_deps branch December 21, 2024 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@siburu siburu siburu approved these changes

@3100 3100 Awaiting requested review from 3100

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yoshidan @siburu