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

deps: update dependencies #25

Merged
merged 2 commits into from
Nov 4, 2021
Merged

deps: update dependencies #25

merged 2 commits into from
Nov 4, 2021

Conversation

0x6b
Copy link
Collaborator

@0x6b 0x6b commented Nov 4, 2021

Update module dependencies, along with container base image for integration test. Will tag this as v1.1.4 once approved and merged.

Closes: #19
Closes: #23
Closes: #24

@0x6b 0x6b self-assigned this Nov 4, 2021
@0x6b 0x6b requested review from moznion and ks6088ts November 4, 2021 07:21
Copy link
Contributor

@moznion moznion left a comment

Choose a reason for hiding this comment

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

@0x6b I'm not sure, but the CI checking raised an error. Could you take a look at that?

@0x6b
Copy link
Collaborator Author

0x6b commented Nov 4, 2021

@moznion thanks for your review. I forgot to update docker image for CI (soratun/soratun-test). Pushed the latest, re-run the job, and CI have passed. Please review the result again. Thanks!

Copy link
Contributor

@moznion moznion left a comment

Choose a reason for hiding this comment

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

Good! Thank you for your contribution.

@moznion moznion merged commit 443cdcd into main Nov 4, 2021
@moznion moznion deleted the update-deps branch November 4, 2021 09:21
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.

Dependency Dashboard
2 participants