Skip to content

Conversation

ZTL-UwU
Copy link
Contributor

@ZTL-UwU ZTL-UwU commented Aug 31, 2025

πŸ”— Linked issue

#1394

❓ Type of change

  • πŸ“– Documentation (updates to the documentation, readme or JSdoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

  1. Fixes registry resolve tests.
  2. Use a valid version of @tabler/icons-vue

Note: the tabler transform test should pass after this pr merges.

πŸ“Έ Screenshots (if appropriate)

image

@Copilot Copilot AI review requested due to automatic review settings August 31, 2025 02:09
Copy link

@Copilot 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 updates test snapshots for registry resolve tests and fixes a package version to use a valid release of @tabler/icons-vue.

  • Updates test snapshots to reflect changes in code formatting (single quotes to double quotes)
  • Upgrades @tabler/icons-vue from version 1.0.0 to 3.34.1

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
packages/cli/test/utils/schema/snapshots/registry-resolve-items-tree.test.ts.snap Updates test snapshots to match new code formatting with double quotes instead of single quotes
apps/www/package.json Upgrades @tabler/icons-vue package to a valid version
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sadeghbarati sadeghbarati merged commit d90434f into unovue:dev Aug 31, 2025
3 of 4 checks passed
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.

2 participants