Skip to content

mirror.yaml: use ubuntu-slim#90

Open
veprbl wants to merge 1 commit intomasterfrom
pr/ubuntu-slim
Open

mirror.yaml: use ubuntu-slim#90
veprbl wants to merge 1 commit intomasterfrom
pr/ubuntu-slim

Conversation

@veprbl
Copy link
Copy Markdown
Member

@veprbl veprbl commented Apr 10, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 10, 2026 19:44
Copy link
Copy Markdown

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 the GitHub Actions workflow that mirrors the repo and triggers EICweb CI by changing the runner label used for the build job.

Changes:

  • Switches the workflow job runner from ubuntu-latest to ubuntu-slim.

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

build:
name: Mirror and Trigger EICweb
runs-on: ubuntu-latest
runs-on: ubuntu-slim
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

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

runs-on: ubuntu-slim is not a valid GitHub-hosted runner label (supported values are ubuntu-latest / ubuntu-24.04 / ubuntu-22.04 / etc.), so this workflow will fail to start. If you intended to use a self-hosted runner, include self-hosted and the label(s) (e.g., runs-on: [self-hosted, ubuntu-slim]); if you intended a slimmer OS/userspace, keep a supported Ubuntu runner and use a container: image like ubuntu:24.04/ubuntu:jammy (or a custom slim image) for the job steps.

Suggested change
runs-on: ubuntu-slim
runs-on: ubuntu-latest

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think this is wrong. Would you agree, @wdconinc ?

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.

Yeah, I think ubuntu-slim here is fine.

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