Skip to content

Conversation

@shiipou
Copy link
Contributor

@shiipou shiipou commented Dec 3, 2025

About this PR

Closes #

Technical highlight/advice

How to test my changes

Checklist

  • I didn't over-scope my PR
  • My PR title matches the commit convention
  • I did not include breaking changes
  • I made my own code-review before requesting one

I included unit tests that cover my changes

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

I added/updated the documentation about my changes

  • 📜 README.md
  • 📕 docs/*.md
  • 📓 docs.lenra.io
  • 🙅 no documentation needed

@shiipou shiipou requested a review from taorepoara December 3, 2025 08:53
@shiipou shiipou self-assigned this Dec 3, 2025
Copilot AI review requested due to automatic review settings December 3, 2025 08:53
Copy link
Contributor

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 enhances the Bun test workflow to support code coverage generation and artifact uploads. The changes enable coverage report generation in lcov format and provide optional artifact upload functionality for the coverage reports.

  • Adds optional artifact-name and artifact-path inputs for coverage report uploads
  • Modifies the test command to generate coverage reports with lcov reporter
  • Implements conditional artifact upload step for coverage reports

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

taorepoara and others added 3 commits December 3, 2025 09:57
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@taorepoara taorepoara merged commit c9d3eae into main Dec 3, 2025
1 check failed
@taorepoara taorepoara deleted the feat-add-coverage-options-to-bun-tests branch December 3, 2025 09:00
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