Skip to content

[cli] refactor error output to use output.error#12012

Merged
kodiakhq[bot] merged 8 commits intomainfrom
erikarowland/zero-2441-refactor-consolelogerror-to-outputerror
Sep 27, 2024
Merged

[cli] refactor error output to use output.error#12012
kodiakhq[bot] merged 8 commits intomainfrom
erikarowland/zero-2441-refactor-consolelogerror-to-outputerror

Conversation

@erikareads
Copy link
Copy Markdown
Contributor

@erikareads erikareads commented Aug 23, 2024

This PR standardizes most remaining console.error(error()) invocations to use output.error instead. handleError is left unchanged in this PR because it's already addressed with the creation of prettyError in #12010.

This PR alters a number of call signatures to pass output into the appropriate functions.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Aug 23, 2024

🦋 Changeset detected

Latest commit: 8154120

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
vercel Patch

Not sure what this means? Click here to learn what changesets are.

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

TooTallNate
TooTallNate previously approved these changes Sep 25, 2024
@kodiakhq kodiakhq bot merged commit 7e8fbd2 into main Sep 27, 2024
@kodiakhq kodiakhq bot deleted the erikarowland/zero-2441-refactor-consolelogerror-to-outputerror branch September 27, 2024 23:00
jeffsee55 pushed a commit that referenced this pull request Oct 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @vercel/[email protected]

### Patch Changes

- add support for `images.localPatterns`
([#12195](#12195))

## [email protected]

### Patch Changes

- Add a command for enabling and disabling telemetry. Telemetry
collection is not currently enabled and when it is, will be a major
version bump for the CLI.
([#12207](#12207))

- [cli] Remove incorrect `--json` flag on `vercel alias`
([#12198](#12198))

- Standardize most error output through `output.error`
([#12012](#12012))

- Replace `psl` with `tldts` for domain parsing
([#12174](#12174))

- add support for `images.localPatterns`
([#12195](#12195))

- [cli] add telemetry tracking to `alias ls`
([#12194](#12194))

- Add support for tracking Continuous Integration vendors with telemetry
([#12180](#12180))

- Add session id to events
([#12179](#12179))

- Updated dependencies
\[[`5431ffd5de6a572f247e63f737576b4a04884f7b`](5431ffd)]:
    -   @vercel/[email protected]
    -   @vercel/[email protected]
    -   @vercel/[email protected]
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

- Replace `@zeit/fetch` usage with `node-fetch`
([#12171](#12171))

- Updated dependencies
\[[`5431ffd5de6a572f247e63f737576b4a04884f7b`](5431ffd)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

-   Link to docs ([#12216](#12216))

## @vercel/[email protected]

### Patch Changes

- Updated dependencies
\[[`5431ffd5de6a572f247e63f737576b4a04884f7b`](5431ffd)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

- add support for `images.localPatterns`
([#12195](#12195))

## @vercel/[email protected]

### Patch Changes

- Updated dependencies
\[[`5431ffd5de6a572f247e63f737576b4a04884f7b`](5431ffd)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

-   Updated dependencies \[]:
    -   @vercel/[email protected]

## @vercel-internals/[email protected]

### Patch Changes

- Add a command for enabling and disabling telemetry. Telemetry
collection is not currently enabled and when it is, will be a major
version bump for the CLI.
([#12207](#12207))

- Updated dependencies
\[[`5431ffd5de6a572f247e63f737576b4a04884f7b`](5431ffd)]:
    -   @vercel/[email protected]

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
QuietCraftsmanship pushed a commit to QuietCraftsmanship/Vercel that referenced this pull request Jul 6, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @vercel/[email protected]

### Patch Changes

- add support for `images.localPatterns`
([#12195](vercel/vercel#12195))

## [email protected]

### Patch Changes

- Add a command for enabling and disabling telemetry. Telemetry
collection is not currently enabled and when it is, will be a major
version bump for the CLI.
([#12207](vercel/vercel#12207))

- [cli] Remove incorrect `--json` flag on `vercel alias`
([#12198](vercel/vercel#12198))

- Standardize most error output through `output.error`
([#12012](vercel/vercel#12012))

- Replace `psl` with `tldts` for domain parsing
([#12174](vercel/vercel#12174))

- add support for `images.localPatterns`
([#12195](vercel/vercel#12195))

- [cli] add telemetry tracking to `alias ls`
([#12194](vercel/vercel#12194))

- Add support for tracking Continuous Integration vendors with telemetry
([#12180](vercel/vercel#12180))

- Add session id to events
([#12179](vercel/vercel#12179))

- Updated dependencies
\[[`9b30d9dd12996b17c743906ba8b59d7dae45930e`](vercel/vercel@9b30d9d)]:
    -   @vercel/[email protected]
    -   @vercel/[email protected]
    -   @vercel/[email protected]
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

- Replace `@zeit/fetch` usage with `node-fetch`
([#12171](vercel/vercel#12171))

- Updated dependencies
\[[`9b30d9dd12996b17c743906ba8b59d7dae45930e`](vercel/vercel@9b30d9d)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

-   Link to docs ([#12216](vercel/vercel#12216))

## @vercel/[email protected]

### Patch Changes

- Updated dependencies
\[[`9b30d9dd12996b17c743906ba8b59d7dae45930e`](vercel/vercel@9b30d9d)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

- add support for `images.localPatterns`
([#12195](vercel/vercel#12195))

## @vercel/[email protected]

### Patch Changes

- Updated dependencies
\[[`9b30d9dd12996b17c743906ba8b59d7dae45930e`](vercel/vercel@9b30d9d)]:
    -   @vercel/[email protected]

## @vercel/[email protected]

### Patch Changes

-   Updated dependencies \[]:
    -   @vercel/[email protected]

## @vercel-internals/[email protected]

### Patch Changes

- Add a command for enabling and disabling telemetry. Telemetry
collection is not currently enabled and when it is, will be a major
version bump for the CLI.
([#12207](vercel/vercel#12207))

- Updated dependencies
\[[`9b30d9dd12996b17c743906ba8b59d7dae45930e`](vercel/vercel@9b30d9d)]:
    -   @vercel/[email protected]

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

5 participants