Skip to content

Replace workspace formatter/linting tool#799

Merged
danditomaso merged 2 commits into
meshtastic:mainfrom
danditomaso:add-workspace-formatter-linter
Aug 22, 2025
Merged

Replace workspace formatter/linting tool#799
danditomaso merged 2 commits into
meshtastic:mainfrom
danditomaso:add-workspace-formatter-linter

Conversation

@danditomaso

Copy link
Copy Markdown
Collaborator

Description

This removes the incorrect devDependency at the workspace root in favour of the correct package from @biomejs

Changes Made

  • Removed incorrect devDependency in workspace
  • Removed @biomejs as devDependency in web repo

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (read
    CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)

Copilot AI review requested due to automatic review settings August 21, 2025 13:11
@vercel

vercel Bot commented Aug 21, 2025

Copy link
Copy Markdown

@danditomaso is attempting to deploy a commit to the Meshtastic Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 standardizes the workspace's linting/formatting tool by replacing an outdated biome package with the official @biomejs/biome package and consolidating it at the workspace root.

  • Replaced the legacy biome package with the official @biomejs/biome package in the root package.json
  • Removed duplicate @biomejs/biome dependency from the web package to avoid redundancy

Reviewed Changes

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

File Description
package.json Updated biome dependency to use official @biomejs/biome package at workspace root
packages/web/package.json Removed duplicate @biomejs/biome dependency to consolidate at workspace level
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.

@danditomaso danditomaso changed the title replaced workspace formatter/linting tool Replace workspace formatter/linting tool Aug 21, 2025
@vercel

vercel Bot commented Aug 22, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
web-test Ready Ready Preview Comment Aug 22, 2025 4:29pm

@danditomaso danditomaso merged commit 1de92cd into meshtastic:main Aug 22, 2025
4 checks passed
@danditomaso danditomaso deleted the add-workspace-formatter-linter branch August 22, 2025 16:30
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