Skip to content

Update PLUGINS.md with workflow setup instructions#211

Merged
lindseywild merged 4 commits intomainfrom
lw/adds-checkout-step
Apr 30, 2026
Merged

Update PLUGINS.md with workflow setup instructions#211
lindseywild merged 4 commits intomainfrom
lw/adds-checkout-step

Conversation

@lindseywild
Copy link
Copy Markdown
Contributor

Added instructions for using the actions/checkout step in workflows, as it's required to run custom plugins.

Added instructions for using the actions/checkout step in workflows.
Copilot AI review requested due to automatic review settings April 30, 2026 14:06
@lindseywild lindseywild requested a review from a team as a code owner April 30, 2026 14:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates PLUGINS.md to document required workflow setup for running custom scanner plugins (notably ensuring the repository contents are available to the action at runtime).

Changes:

  • Adds guidance to include an actions/checkout step before the scanner action.
  • Adds a workflow snippet illustrating the required step ordering.
Show a summary per file
File Description
PLUGINS.md Documents adding actions/checkout before the scanner step, with an example workflow snippet.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 3

Comment thread PLUGINS.md Outdated
Comment thread PLUGINS.md Outdated
Comment thread PLUGINS.md Outdated
lindseywild and others added 2 commits April 30, 2026 10:27
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread PLUGINS.md Outdated
- Each plugin should have one `index.ts` OR `index.js` file inside its folder.
- The `index.ts/index.js` file must export a `name` field. This is the name used to pass to the `scans` input. So if the plugin exports a name value of `my-custom-plugin` and we pass the following to the scanner action inputs: `scans: ['my-custom-plugin']`, it would cause the scanner to only run that plugin.
- The `index.ts/index.js` file must export a default function. This is the function that the scanner uses to run the plugin. This can be an async function.
- In your workflow file, before the scanner step, add `- uses: actions/checkout@v6` (or whatever the current version is), like so:
Copy link
Copy Markdown
Contributor

@abdulahmad307 abdulahmad307 Apr 30, 2026

Choose a reason for hiding this comment

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

I think it would be valuable to include why this is necessary. Someone might skip this step not knowing why its required (and thinking its not important), and then not know why their custom plugin isn't loading.

Clarify instructions for adding plugins to the scanner.
@lindseywild lindseywild merged commit 95475e6 into main Apr 30, 2026
6 checks passed
@lindseywild lindseywild deleted the lw/adds-checkout-step branch April 30, 2026 17:15
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.

3 participants