Skip to content

Address v3.4.1 release review feedback - #254

Closed
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-plan-friday-releases
Closed

Address v3.4.1 release review feedback#254
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-plan-friday-releases

Conversation

@taarikashenafi

Copy link
Copy Markdown
Contributor

Summary

Addresses the documentation and action-pin feedback from the v3.4.1 release PR (#253):

  • calls out scanner v3.4.1 as the minimum version for NPM plugin loading
  • moves the minimum version requirement to the top of the plugin documentation
  • annotates the pinned ruby/setup-ruby SHA with its release version

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 8bad6e4b-711c-4c01-9354-2e4a9890efe1
Copilot AI review requested due to automatic review settings July 29, 2026 22:11
@taarikashenafi
taarikashenafi requested a review from a team as a code owner July 29, 2026 22:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Clarifies the minimum scanner version for NPM plugins and documents the pinned Ruby action version.

Changes:

  • Highlights scanner v3.4.1 as the minimum for NPM plugin loading.
  • Annotates the pinned ruby/setup-ruby SHA with its matching release.
Show a summary per file
File Description
PLUGINS.md Promotes the minimum-version requirement.
.github/actions/find/README.md Adds the requirement to scans documentation.
.github/workflows/test.yml Documents the pinned action version.

Review details

Tip

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

  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Medium

@taarikashenafi

Copy link
Copy Markdown
Contributor Author

Closing because the release follow-up was handled directly on the original release PR.

@taarikashenafi
taarikashenafi deleted the taarikashenafi-plan-friday-releases branch July 29, 2026 22:41
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