Skip to content

fix: inline sourcemaps to help with debugging - #6768

Merged
matthewkeil merged 1 commit into
electra-forkfrom
mkeil/inline-maps-for-vitest-electra
May 14, 2024
Merged

fix: inline sourcemaps to help with debugging#6768
matthewkeil merged 1 commit into
electra-forkfrom
mkeil/inline-maps-for-vitest-electra

Conversation

@matthewkeil

@matthewkeil matthewkeil commented May 14, 2024

Copy link
Copy Markdown
Member

Motivation

Turns on inline sourcemaps to help with debugging under vitest. Only works with vscode enabled debugging console and will work to get this expended to a debug launch profile

Screenshot 2024-05-14 at 10 42 43 AM

@matthewkeil
matthewkeil requested a review from a team as a code owner May 14, 2024 07:41
@matthewkeil
matthewkeil merged commit 8065222 into electra-fork May 14, 2024
@matthewkeil
matthewkeil deleted the mkeil/inline-maps-for-vitest-electra branch May 14, 2024 08:42
Comment thread vite.base.config.ts
esbuild: {
banner,
legalComments: "none",
sourcemap: "inline",

@nazarhussain nazarhussain May 24, 2024

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.

The vite file is not for the vitest, rather it's for the bundle packages. I am surprised changing any of this effects debugging. If you actually need sourcemaps for the tests then use this option https://vitest.dev/config/#server-sourcemap

g11tech pushed a commit to g11tech/lodestar that referenced this pull request Jun 19, 2024
@wemeetagain

Copy link
Copy Markdown
Member

馃帀 This PR is included in v1.22.0 馃帀

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.

4 participants