Skip to content

feat(ci): add unit test step to build workflow and update package scripts, fix broken vite test#148

Open
Scott McMaster (scottmcmaster) wants to merge 1 commit into
developfrom
scott/ci-unit-tests
Open

feat(ci): add unit test step to build workflow and update package scripts, fix broken vite test#148
Scott McMaster (scottmcmaster) wants to merge 1 commit into
developfrom
scott/ci-unit-tests

Conversation

@scottmcmaster
Copy link
Copy Markdown
Contributor

Summary

We've had a long-outstanding item I entered saying that we weren't running unit tests in CI. I fixed them and inserted them after the main tauri app build step.

Test Plan

Run in CI.

Docs

  • Docs updated (companion PR in darkmatter/nixmac-web: #___)
  • No docs update needed

@github-actions
Copy link
Copy Markdown
Contributor

Warnings
⚠️ Please assign this PR to someone (usually yourself).
⚠️

package.json changed but bun.lock did not. Run bun install and commit the lockfile.

Messages
📖

🤖 This PR touches CI / infra — reviewers please pay extra attention:

  • .github/workflows/build.yaml

📋 PR Overview

Lines changed 25 (+17 / -8)
Files 0 added, 4 modified, 0 deleted
Draft / WIP no
Has Test Plan yes
New UI components no
New Storybook stories no
New Rust modules no
New TS source files no
New tests no
package.json touched yes
Cargo.toml touched no
Infra / CI touched yes

🔬 Coverage

Report Lines Statements Functions Branches
apps/native/coverage/coverage-summary.json 17.3% 17.3% 27.8% 52.9%

Generated by 🚫 dangerJS against 747d585

@github-actions
Copy link
Copy Markdown
Contributor

nixmac Peekaboo E2E: ❌ FAIL

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.

1 participant