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: fix test flow to install libsodium #115

Merged
merged 4 commits into from
Nov 22, 2022
Merged

Conversation

ulbqb
Copy link
Member

@ulbqb ulbqb commented Nov 18, 2022

Description

This PR has following changes.

  • add test.yml to diff target in test.yml
  • add step to install libsodium

related: #117

Motivation and context

Even if ci test is changed, the test is skipped. I missed that it had to install libsodium because of that.

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

@ulbqb ulbqb self-assigned this Nov 18, 2022
@ulbqb ulbqb marked this pull request as ready for review November 21, 2022 04:42
@ulbqb ulbqb merged commit 44dd664 into main Nov 22, 2022
@ulbqb ulbqb deleted the fix/libsodium_installing branch November 22, 2022 03:50
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.

3 participants