fix: never auto-open the HTML report, document npm init mobilewright@latest - #17
Conversation
…latest
- generated config uses reporter: [['html', { open: 'never' }]] so a failing
first run doesn't serve the report and block the terminal
- .gitignore also covers /mobilewright-report/
- README: npm init mobilewright@latest
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (3)
Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour. WalkthroughThe README now uses Priority: ➖ Normal Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The scaffolding changes are ready to merge with no identified user-facing or operational risk. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
reporter: 'html'a failing run in a terminal serves the report, opens the browser and waits for Ctrl-C. The generated config now usesreporter: [['html', { open: 'never' }]]. Compatible with fix: rebrand the HTML report as Mobilewright end to end mobilewright#324, which routes confightmlreporters through the branded reporter and keepsopen..gitignorealso gets/mobilewright-report/(the report folder after mobilewright#324), alongside/playwright-report/for older mobilewright releases.npm init mobilewright@latest, so users don't run an older scaffolder cached by npx. Yarn and pnpm lines unchanged, same as Playwright's README.Test plan
npm test(53): new test for the reporter line; gitignore tests updatednpm run lint,npm run buildmobilewright test --listandtsc --noEmitpass