Skip to content

Fixes actions test summary#7158

Merged
MartinM85 merged 1 commit intopnp:mainfrom
milanholemans:fix-workflow
Mar 7, 2026
Merged

Fixes actions test summary#7158
MartinM85 merged 1 commit intopnp:mainfrom
milanholemans:fix-workflow

Conversation

@milanholemans
Copy link
Contributor

Noticed that our PR check workflow was throwing errors:

image

Copilot AI review requested due to automatic review settings March 5, 2026 21:43
Copy link

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

This PR fixes the GitHub Actions test summary reporter used by Mocha so it no longer errors when writing to GITHUB_STEP_SUMMARY during CI runs.

Changes:

  • Defers loading @actions/core until writeSummary() and awaits the dynamic import.
  • Removes eager initialization of summary to avoid referencing core.summary before the module is available.

@MartinM85 MartinM85 self-assigned this Mar 7, 2026
@MartinM85 MartinM85 merged commit 8ee601a into pnp:main Mar 7, 2026
14 checks passed
@milanholemans milanholemans deleted the fix-workflow branch March 7, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants