Skip to content

Add CI for vmm-ui#426

Merged
kvinwang merged 3 commits into
masterfrom
ci-for-vmm-ui
Jan 5, 2026
Merged

Add CI for vmm-ui#426
kvinwang merged 3 commits into
masterfrom
ci-for-vmm-ui

Conversation

@kvinwang

@kvinwang kvinwang commented Jan 5, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread .github/workflows/vmm-ui.yml Fixed
Comment on lines +15 to +38
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Install Node.js
uses: actions/setup-node@v4
with:
node-version: '20'

- name: Build vmm UI
run: |
npm ci
npm run build
working-directory: vmm/ui

- name: Ensure vmm UI build is clean
run: |
if [ -n "$(git status --porcelain)" ]; then
echo "VMM UI build produced uncommitted changes."
echo "Run: cd vmm/ui && npm ci && npm run build"
echo "Then commit the updated build output `vmm/src/console_v1.html`"
git status --porcelain
exit 1
fi

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}

Copilot Autofix

AI 7 months ago

Copilot could not generate an autofix suggestion

Copilot could not generate an autofix suggestion for this alert. Try pushing a new commit or if the problem persists contact support.

@kvinwang
kvinwang enabled auto-merge January 5, 2026 01:23
@kvinwang
kvinwang merged commit cb2eddd into master Jan 5, 2026
11 checks passed
kvinwang added a commit that referenced this pull request Jan 19, 2026
@kvinwang
kvinwang deleted the ci-for-vmm-ui branch January 21, 2026 06:22
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