Skip to content

build: update actions and Node.js version for test matrix#108

Merged
lirantal merged 1 commit intomainfrom
lirantal-patch-3
Apr 2, 2025
Merged

build: update actions and Node.js version for test matrix#108
lirantal merged 1 commit intomainfrom
lirantal-patch-3

Conversation

@lirantal
Copy link
Owner

@lirantal lirantal commented Apr 2, 2025

User description

Description

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist:

  • I have updated the documentation (if required).
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I added a picture of a cute animal cause it's fun

PR Type

Enhancement, Configuration changes


Description

  • Updated Node.js versions in the test matrix.

  • Upgraded GitHub Actions versions for checkout and setup-node.

  • Adjusted workflows to align with updated configurations.


Changes walkthrough 📝

Relevant files
Configuration changes
main.yml
Update Node.js versions and GitHub Actions versions           

.github/workflows/main.yml

  • Updated Node.js versions in the matrix to 18, 20, and 22.
  • Upgraded actions/checkout to version v4.
  • Upgraded actions/setup-node to version v4.
  • Ensured compatibility with updated configurations.
  • +5/-5     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @qodo-free-for-open-source-projects

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-free-for-open-source-projects

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @codecov-commenter
    Copy link

    Codecov Report

    All modified and coverable lines are covered by tests ✅

    Project coverage is 89.60%. Comparing base (6f337ba) to head (b3b5b8d).

    Additional details and impacted files
    @@           Coverage Diff           @@
    ##             main     #108   +/-   ##
    =======================================
      Coverage   89.60%   89.60%           
    =======================================
      Files           5        5           
      Lines         202      202           
      Branches       63       63           
    =======================================
      Hits          181      181           
      Misses         20       20           
      Partials        1        1           

    ☔ View full report in Codecov by Sentry.
    📢 Have feedback on the report? Share it here.

    🚀 New features to boost your workflow:
    • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
    • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

    @lirantal lirantal merged commit ab3eaa6 into main Apr 2, 2025
    9 checks passed
    @github-actions
    Copy link

    github-actions bot commented Sep 4, 2025

    🎉 This PR is included in version 1.14.11 🎉

    The release is available on:

    Your semantic-release bot 📦🚀

    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.

    2 participants