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.

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

Feature: replace .vscode directory with a .code-workspace file #106

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

andy-blum
Copy link
Contributor

The Problem/Issue/Bug

The .vscode directory stores files that can be used to configure the editor's settings, debugger, et cetera, but the same thing and more can be accomplished in a .code-workspace file. Additionally, the code-workspace file allows users to create "multi-root workspaces" which can make navigating code easier when applications are spread across large numbers of files.

How this PR Solves The Problem

This PR moves the existing launch.json and settings.json files into the code workspace and sets three workspace folders:

  • DrupalPod (/workspace/DrupalPod/)
  • Drupal Core (/workspace/DrupalPod/repos/drupal/)
  • Web Root (/workspace/DrupalPod/web)

Setting the folders up this way let us remove the distractions of other files/folders when we want to focus on work in the webroot or drupal git repo.

Manual Testing Instructions

Open the deploy preview, which should automatically set the editor's workspace to use the code-workspace file.

Related Issue Link(s)

None

Release/Deployment notes

@codesandbox
Copy link

codesandbox bot commented Nov 22, 2022

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

@ghost
Copy link

ghost commented Nov 22, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

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.

None yet

1 participant