Skip to content

Conversation

@hetangmodi-crest
Copy link
Contributor

@hetangmodi-crest hetangmodi-crest commented Oct 22, 2025

Issue number: -

PR Type

What kind of change does this PR introduce?

  • Feature
  • Bug Fix
  • Refactoring (no functional or API changes)
  • Documentation Update
  • Maintenance (dependency updates, CI, etc.)

Summary

Changes

Updated the vite version to v7.1.12 as mentioned by Fossa failure. Cleaned up dependencies in yarn.lock using yarn-deduplicate tool.
Also, explicitly using node version 22.21.0

User experience

No change, just the vite dependency has been updated.

Checklist

If an item doesn't apply to your changes, leave it unchecked.

Review

  • self-review - I have performed a self-review of this change according to the development guidelines
  • Changes are documented. The documentation is understandable, examples work (more info)
  • PR title and description follows the contributing principles
  • meeting - I have scheduled a meeting or recorded a demo to explain these changes (if there is a video, put a link below and in the ticket)

Tests

See the testing doc.

  • Unit - tests have been added/modified to cover the changes
  • Smoke - tests have been added/modified to cover the changes
  • UI - tests have been added/modified to cover the changes
  • coverage - I have checked the code coverage of my changes (see more)

Demo/meeting:

Reviewers are encouraged to request meetings or demos if any part of the change is unclear

@hetangmodi-crest hetangmodi-crest self-assigned this Oct 22, 2025
@hetangmodi-crest hetangmodi-crest added the dependencies Pull requests that update a dependency file label Oct 22, 2025
@hetangmodi-crest hetangmodi-crest marked this pull request as ready for review October 29, 2025 09:12
@hetangmodi-crest hetangmodi-crest requested a review from a team as a code owner October 29, 2025 09:12
rohanm-crest
rohanm-crest previously approved these changes Oct 30, 2025
soleksy-splunk
soleksy-splunk previously approved these changes Oct 30, 2025
Copy link
Contributor

@soleksy-splunk soleksy-splunk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Copy link
Contributor

@soleksy-splunk soleksy-splunk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@hetangmodi-crest hetangmodi-crest merged commit 7b10fae into develop Oct 31, 2025
143 of 145 checks passed
@hetangmodi-crest hetangmodi-crest deleted the chore/resolve-vite-vuln branch October 31, 2025 05:33
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants