Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(gatsby): Enable source maps when compiling Gatsby files in development #36450

Merged
merged 5 commits into from
Sep 20, 2022
Merged

feat(gatsby): Enable source maps when compiling Gatsby files in development #36450

merged 5 commits into from
Sep 20, 2022

Conversation

Auspicus
Copy link
Contributor

@Auspicus Auspicus commented Aug 23, 2022 •
edited by LekoArts
Loading

Description

This change enables source maps to be output during compilation of Gatsby files in development enabling better debugger integration.

Documentation

  • Confirm that source maps generated for Gatsby pages, templates, functions and components map into the debugger
    • src/pages
    • src/templates
    • src/components
    • src/api
  • Generate source maps for Gatsby files
  • Update docs to note that PARCEL_WORKERS=0 and GATSBY_CPU_COUNT=1 should be passed as env vars when debugging

Related Issues

Fixes #36448

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 23, 2022
@Auspicus Auspicus marked this pull request as draft August 23, 2022 05:33
@imjoshin imjoshin added topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Aug 24, 2022
@Auspicus Auspicus mentioned this pull request Sep 20, 2022
Closed
@Auspicus Auspicus marked this pull request as ready for review September 20, 2022 01:35
LekoArts
LekoArts approved these changes Sep 20, 2022
View reviewed changes
Copy link
Contributor

@LekoArts LekoArts left a comment

Choose a reason for hiding this comment

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

Thanks!

@LekoArts LekoArts merged commit a9b2de8 into gatsbyjs:master Sep 20, 2022
@snyk-bot snyk-bot mentioned this pull request Sep 28, 2022
Open
@kaocher82 kaocher82 mentioned this pull request Sep 28, 2022
Open
@snyk-bot snyk-bot mentioned this pull request Sep 28, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LekoArts LekoArts LekoArts approved these changes

Assignees
No one assigned
Labels
topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[gatsby] How to use VS Code debugger with Gatsby
3 participants
@Auspicus @LekoArts @imjoshin