-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
[FileLocksmith] Switch IPC between ctx menu dll and UI from writing to file to pipe #31556
base: main
Are you sure you want to change the base?
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.
Gave it a test, but restarting as administrator seems to break this change. Perhaps that's one of the reasons why it used a file in the first place? 🤔
Higher priorities at this moment. Changing it to a draft for the time being. |
@stefansjfw how is this going? 🙂 |
Still on hold. What Jaime said in the comment above... :) |
Summary of the Pull Request
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed
Tested local and release CI build