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

Add Mikado hook as a featured import method for SDVX #947

Merged
merged 2 commits into from
Sep 28, 2023

Conversation

adamaq01
Copy link
Contributor

@adamaq01 adamaq01 commented Sep 7, 2023

Here's a link of the hook's source code: https://github.com/adamaq01/mikado

On the discord it seemed that some people were interested in it being featured so here's the PR.

There are some places where I am not sure what to put (for example the description of the hook, since theres also Barbatos) so I put some TODOs in there. It would be great to discuss about what these should be.
Same thing for the position on the webpage etc...
Marking it as a draft until this is resolved :)

@zkrising
Copy link
Owner

If this hook supercedes barbatos then maybe you can inherit all of the barbatos descriptions, and barbatos can be marked as deprecated/for older versions of sdvx? I'm not up to date with what works on what anymore. That's what I'd do anyway.

Neat that the source code for this is public. Very interesting to see the rust guts of hooking into DLLs like this :P

@adamaq01 adamaq01 marked this pull request as ready for review September 28, 2023 03:09
@zkrising zkrising merged commit f4292dc into zkrising:staging Sep 28, 2023
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