-
Notifications
You must be signed in to change notification settings - Fork 201
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
finalize 2.15.0 #1331
finalize 2.15.0 #1331
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@hitenkoku Sorry, could you check why this is failing? https://github.com/Yamato-Security/hayabusa/actions/runs/8761069797 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be caused by the following changes :(
(It was probably compiled with 0.11.0
before, and now it's compiled with 0.10.x
)
- https://github.com/console-rs/dialoguer/blob/master/CHANGELOG.md#breaking
- Remove
ColorfulTheme.inline_selections
console-rs/dialoguer#277
In my environment, the issue was resolved by upgrading to version 0.11 or higher as shown below!
- https://github.com/Yamato-Security/hayabusa/blob/finalize-2.15.0/Cargo.toml#L22
dialoguer = "0.11.*"
@fukusuket Thanks! It still wasn't working just by updating dialoguer in Hayabusa for me but when I updated it in hayabusa-evtx it now compiles without any error. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hayabusa evtx also uses dialoger, so this is probably the cause! I think it's better to update this as well:) LGTM🚀
@hitenkoku @fukusuket I decided to release 2.15.0 with the
windash
support to give people time to upgrade before we stop doing thewindash
conversion on the backend.