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

Update msgpckr to 1.10.2 #13722

Merged
merged 1 commit into from
May 27, 2024
Merged

Conversation

tsmaeder
Copy link
Contributor

What it does

Fixes #13532

The hex editor was not loading in the end due to kriszyp/msgpackr#135. The issue was fixed and released in a new version.

How to test

Install the hex editor and make sure you can "Open with" files in the hex editor and the contents show correctly. Since this updates our message packing lib, observe the logs to make sure we don't get any new suspicious logs.

Follow-ups

Review checklist

Reminder for reviewers

Fixes eclipse-theia#13532

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
@tsmaeder tsmaeder requested a review from tortmayr May 20, 2024 14:27
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

LGTM. The hex editor works as expected for me.

@tsmaeder tsmaeder merged commit c66bf7a into eclipse-theia:master May 27, 2024
14 checks passed
@jfaltermeier jfaltermeier added this to the 1.50.0 milestone Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

VS HexEditor pane comes up blank
3 participants