Skip to content

Set up node for Copilot - #25

Merged
MagnusIntel merged 1 commit into
mainfrom
emaus/copilot-config
Nov 6, 2026
Merged

MagnusIntel merged 1 commit into
mainfrom
emaus/copilot-config

Conversation

@emanuelen5

Copy link
Copy Markdown
Contributor

That way it doesn't have to figure out by itself that it should install it

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues were identified in the workflow changes.

No open issues 🎉

What changed in this PR

This pull request prepares the Copilot code-review workflow with Node.js 24 and the repository’s locked npm dependencies.

Changes:

  • Sets up Node.js 24 using the repository’s pinned actions/setup-node action.
  • Installs dependencies reproducibly with npm ci.
  • Disables package-manager caching for the setup step.
File Description
.github/​workflows/​copilot-code-review.yml Adds Node.js setup and dependency installation to the Copilot workflow.

Review effort: Lite

@emanuelen5
emanuelen5 marked this pull request as ready for review November 6, 2026 09:18
@emanuelen5
emanuelen5 requested a review from a team as a code owner November 6, 2026 09:18
That way it doesn't have to figure out by itself that it should install it

Signed-off-by: Erasmus Cedernaes <erasmus.cedernaes@intel.com>
@emanuelen5
emanuelen5 force-pushed the emaus/copilot-config branch from d098181 to 19fe48a Compare November 6, 2026 09:19

@MagnusIntel MagnusIntel 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.

Nice

@MagnusIntel
MagnusIntel merged commit b5377bf into main Nov 6, 2026
6 checks passed
@MagnusIntel
MagnusIntel deleted the emaus/copilot-config branch November 6, 2026 10:33
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