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

starter plugin.json #139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

starter plugin.json #139

wants to merge 1 commit into from

Conversation

psifertex
Copy link

BN supports having plugins in the plugin manager that don't automatically install but simply link to the original repository. This can help with discoverability. I've populated a sample plugin.json for you, but feel free to adjust it as desired.

Copy link

google-cla bot commented Oct 1, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@psifertex
Copy link
Author

Tried to update my CLA with my GH username so hopefully that works now?

@cblichmann
Copy link
Member

Thanks, Jordan! Does this need to live in the repo root, though? plugin.json is a rather generic name, so I'd prefer if we could put it in binaryninja/.

@psifertex
Copy link
Author

Oh, good point, yeah that should work fine (though I haven't double checked that with a plugin yet, it should work still). If you want to make the change or just do your own file go for it, otherwise I can amend this.

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