Skip to content

Comments

chore: update some dependencies#822

Open
rossiam wants to merge 1 commit intoSmartThingsCommunity:mainfrom
rossiam:update-deps
Open

chore: update some dependencies#822
rossiam wants to merge 1 commit intoSmartThingsCommunity:mainfrom
rossiam:update-deps

Conversation

@rossiam
Copy link
Collaborator

@rossiam rossiam commented Feb 23, 2026

Update several non-jest and non-eslint dependencies.

The three exceptions:

  • @inquirer/prompts - includes breaking changes that require a bit more looking into
  • ora - breaks final binary; I'm not sure why yet
  • env-paths
    • has a breaking change not listed in its release notes that we can't work around without moving the config directory
    • I'm not sure what the path forward on this one is yet but the library is pretty simple so worst case we could just incorporate it and make our own changes.
    • The breaking change is that they no longer allow the config directory to be nested below the default location. (Had this been a requirement when I switched to this library and moved some config directories anyway, I would have used something like smartthings-cli instead of the @smartthings/cli which I used because it matches the old oclif path more closely. I won't force users to switch to a new directory again.)
    • Breaking change
    • Library doing the new validation

@rossiam rossiam requested a review from a team February 23, 2026 21:14
@changeset-bot
Copy link

changeset-bot bot commented Feb 23, 2026

⚠️ No Changeset found

Latest commit: f48b928

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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