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

Snissn/pinmgrtestpanic #650

Merged
merged 2 commits into from
Dec 4, 2022
Merged

Snissn/pinmgrtestpanic #650

merged 2 commits into from
Dec 4, 2022

Conversation

snissn
Copy link
Contributor

@snissn snissn commented Dec 4, 2022

fix for pin mgr panic by stashing addr in a string based datastructure before serializing

@snissn snissn enabled auto-merge December 4, 2022 04:14
@snissn snissn requested a review from Zorlin December 4, 2022 04:16
Copy link
Contributor

@Zorlin Zorlin left a comment

Choose a reason for hiding this comment

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

LGTM, merge if tests pass and build works.

@snissn snissn merged commit b885c4d into dev Dec 4, 2022
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.

2 participants