Skip to content

[cli] remove unused source map pkg#10577

Merged
kodiakhq[bot] merged 1 commit intomainfrom
remove-zeit-source-map-support
Sep 23, 2023
Merged

[cli] remove unused source map pkg#10577
kodiakhq[bot] merged 1 commit intomainfrom
remove-zeit-source-map-support

Conversation

@styfle
Copy link
Copy Markdown
Member

@styfle styfle commented Sep 23, 2023

This PR removes a deprecated package that was originally added in PR #1205

The comment said:

// we only enable source maps while developing, since
// they have a small performance hit. for this, we
// look for `pkg`, which is only present in the final bin

Some time along the way this was changed from development only, to always being installed.

However, no longer need this package because we can enable source maps for development by using --enable-source-map which has been available since Node.js 12

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 23, 2023

🦋 Changeset detected

Latest commit: 9e6b20c

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

Copy link
Copy Markdown

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@kodiakhq kodiakhq bot merged commit c147ead into main Sep 23, 2023
@kodiakhq kodiakhq bot deleted the remove-zeit-source-map-support branch September 23, 2023 03:25
Ethan-Arrowood pushed a commit that referenced this pull request Oct 3, 2023
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@32.4.0

### Minor Changes

- Restore unsetting teamId for non-team accounts
([#10612](#10612))

### Patch Changes

- remove unused source map pkg
([#10577](#10577))

- disable source map for prod build
([#10575](#10575))

- Better rendering upon authentication error in `vc cert ls`
([#10551](#10551))

- Updated dependencies
\[[`e9026c7a6`](e9026c7),
[`ea5bc8806`](ea5bc88),
[`a4996e1c5`](a4996e1),
[`a18ed98f2`](a18ed98),
[`2f5b0aeeb`](2f5b0ae),
[`09f1bbfa4`](09f1bbf),
[`ce7e82fa7`](ce7e82f)]:
    -   @vercel/next@4.0.9
    -   @vercel/go@3.0.3
    -   @vercel/build-utils@7.2.2
    -   @vercel/node@3.0.7
    -   @vercel/redwood@2.0.4
    -   @vercel/remix-builder@2.0.9
    -   @vercel/static-build@2.0.8

## @vercel/build-utils@7.2.2

### Patch Changes

- [cli] Update bun detection and add tests for projects with both
bunlock binary and yarn.lock text files
([#10583](#10583))

## @vercel/client@13.0.5

### Patch Changes

- Updated dependencies
\[[`2f5b0aeeb`](2f5b0ae)]:
    -   @vercel/build-utils@7.2.2

## @vercel/fs-detectors@5.1.1

### Patch Changes

- [cli] Update bun detection and add tests for projects with both
bunlock binary and yarn.lock text files
([#10583](#10583))

## @vercel/gatsby-plugin-vercel-builder@2.0.7

### Patch Changes

- Updated dependencies
\[[`2f5b0aeeb`](2f5b0ae)]:
    -   @vercel/build-utils@7.2.2

## @vercel/go@3.0.3

### Patch Changes

- Update broken documentation link
([#10579](#10579))

## @vercel/next@4.0.9

### Patch Changes

- Correct output file tracing and limit calculation
([#10631](#10631))

- Fix the instrumentation hook on Next.js Edge Functions
([#10608](#10608))

- [next] fix lambda creation for i18n edge pages
([#10630](#10630))

- Revert "[next][node][redwood][remix] Update @vercel/nft (#10540)"
([#10633](#10633))

- Update `@vercel/nft` to 0.24.2
([#10644](#10644))

## @vercel/node@3.0.7

### Patch Changes

- Revert "[next][node][redwood][remix] Update @vercel/nft (#10540)"
([#10633](#10633))

- Update `@vercel/nft` to 0.24.2
([#10644](#10644))

- Updated dependencies
\[[`2f5b0aeeb`](2f5b0ae)]:
    -   @vercel/build-utils@7.2.2

## @vercel/redwood@2.0.4

### Patch Changes

- Revert "[next][node][redwood][remix] Update @vercel/nft (#10540)"
([#10633](#10633))

- Update `@vercel/nft` to 0.24.2
([#10644](#10644))

## @vercel/remix-builder@2.0.9

### Patch Changes

- Revert "[next][node][redwood][remix] Update @vercel/nft (#10540)"
([#10633](#10633))

- Update `@vercel/nft` to 0.24.2
([#10644](#10644))

## @vercel/static-build@2.0.8

### Patch Changes

-   Updated dependencies \[]:
    -   @vercel/gatsby-plugin-vercel-builder@2.0.7

## @vercel-internals/types@1.0.12

### Patch Changes

- Updated dependencies
\[[`2f5b0aeeb`](2f5b0ae)]:
    -   @vercel/build-utils@7.2.2

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants