Skip to content
This repository has been archived by the owner on May 30, 2024. It is now read-only.

Use simple object for debugConfiguration.targetFolder #992

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

st0012
Copy link
Member

@st0012 st0012 commented Jan 18, 2024

Motivation

Because debugConfiguration can be serialized/deserialized, debugConfiguration.targetFolder.uri may be converted into a plain JS object and loses the fsPath property and causes cwd to be undefined when working with a remote workspace.

Implementation

This commit fixes the issue by storing the fsPath property directly in debugConfiguration.targetFolder.

Automated Tests

Manual Tests

It's been manually tested on Spin.

Because `debugConfiguration` can be serialized/deserialized,
`debugConfiguration.targetFolder.uri` may be converted into a plain JS
object and loses the `fsPath` property and causes `cwd` to be undefined
when working with a remote workspace.

This commit fixes the issue by storing the `fsPath` property directly
in `debugConfiguration.targetFolder`.

Co-authored-by: Vinicius Stock <[email protected]>
@st0012 st0012 added the bugfix This PR will fix an existing bug label Jan 18, 2024
@st0012 st0012 requested a review from a team as a code owner January 18, 2024 18:11
@st0012 st0012 requested review from Morriar and vinistock January 18, 2024 18:11
@st0012 st0012 merged commit e458b3a into main Jan 19, 2024
10 checks passed
@st0012 st0012 deleted the fix-undefined-cwd branch January 19, 2024 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix This PR will fix an existing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants