Demo project as a submodule

.github/workflows/continuous-integration.yml

@@ -18,6 +18,7 @@ jobs:
         uses: actions/checkout@v3
         with:
           fetch-depth: 0
+          submodules: true
 
       - name: Setup Python environment
         uses: actions/setup-python@v4

.github/workflows/cross-platform-test.yml

@@ -21,6 +21,7 @@ jobs:
         uses: actions/checkout@v3
         with:
           fetch-depth: 0
+          submodules: true
 
       - name: Setup Node.js environment
         uses: actions/setup-node@v3

.github/workflows/dvc-cli-output-test.yml

@@ -16,6 +16,7 @@ jobs:
         uses: actions/checkout@v3
         with:
           fetch-depth: 0
+          submodules: true
 
       - name: Setup Node.js environment
         uses: actions/setup-node@v3

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "demo"]
+	path = demo
+	url = https://github.com/iterative/vscode-dvc-demo

extension/package.json

@@ -1412,7 +1412,7 @@
             },
             "completionEvents": [
               "onLink:https://github.com/iterative/example-get-started",
-              "onLink:https://github.com/iterative/vscode-dvc/tree/main/demo",
+              "onLink:https://github.com/iterative/vscode-dvc-demo",
               "onLink:https://dvc.org/doc/dvclive"
             ]
           },

extension/resources/walkthrough/setup-project.md

@@ -5,8 +5,8 @@
 
 💡 Check out the
 [DVC Get Started](https://github.com/iterative/example-get-started) or
-[Extension Demo](https://github.com/iterative/vscode-dvc/tree/main/demo)
-projects to quickly try the extension.
+[Extension Demo](https://github.com/iterative/vscode-dvc-demo) projects to
+quickly try the extension.
 
 To quickly setup a new DVC project run
 [`dvc exp init -i`](https://dvc.org/doc/command-reference/exp/init#example-interactive-mode)

extension/src/cli/git/constants.ts

@@ -1,11 +1,13 @@
 import { join } from 'path'
 
-export const DOT_GIT = '.git'
-export const DOT_GIT_HEAD = join(DOT_GIT, 'HEAD')
-export const DOT_GIT_INDEX = join(DOT_GIT, 'index')
-export const GIT_REFS = join(DOT_GIT, 'refs')
-export const GIT_LOGS_REFS = join(DOT_GIT, 'logs', 'refs')
-export const HEADS_GIT_REFS = join(GIT_REFS, 'heads')
+export const gitPath = {
+  DOT_GIT: '.git',
+  DOT_GIT_HEAD: 'HEAD',
+  DOT_GIT_INDEX: 'index',
+  GIT_LOGS_REFS: join('logs', 'ref'),
+  GIT_REFS: 'refs',
+  HEADS_GIT_REFS: 'heads'
+} as const
 
 export enum Command {
   ADD = 'add',

extension/src/cli/git/executor.test.ts

@@ -4,6 +4,7 @@ import { GitExecutor } from './executor'
 import { createProcess } from '../../processExecution'
 import { CliResult, CliStarted } from '..'
 import { getMockedProcess } from '../../test/util/jest'
+import { standardizePath } from '../../fileSystem/path'
 
 jest.mock('vscode')
 jest.mock('@hediet/std/disposable')
@@ -38,9 +39,10 @@ describe('GitExecutor', () => {
     } as unknown as EventEmitter<CliStarted>
   })
 
+  const cwd = standardizePath(__dirname) as string
+
   describe('pushBranch', () => {
     it('should call createProcess with the correct parameters to push a branch', async () => {
-      const cwd = __dirname
       const branchName = 'my-branch'
       mockedCreateProcess.mockReturnValueOnce(
         getMockedProcess(
@@ -57,7 +59,6 @@ describe('GitExecutor', () => {
     })
 
     it('should call createProcess with the correct parameters to push the current branch', async () => {
-      const cwd = __dirname
       mockedCreateProcess.mockReturnValueOnce(
         getMockedProcess('Everything up-to-date')
       )
@@ -73,7 +74,6 @@ describe('GitExecutor', () => {
 
   describe('stageAndCommit', () => {
     it('should call createProcess with the correct parameters to stage all files and then commit', async () => {
-      const cwd = __dirname
       const message = 'best experiment'
       mockedCreateProcess.mockReturnValueOnce(getMockedProcess(cwd))
       mockedCreateProcess

extension/src/cli/git/index.ts

@@ -1,11 +1,12 @@
 import { Command, Flag } from './constants'
 import { getOptions } from './options'
 import { Cli } from '..'
+import { standardizePath } from '../../fileSystem/path'
 
 export class GitCli extends Cli {
-  public getGitRepositoryRoot(cwd: string) {
+  public async getGitRepositoryRoot(cwd: string) {
     const options = getOptions(cwd, Command.REV_PARSE, Flag.SHOW_TOPLEVEL)
 
-    return this.executeProcess(options)
+    return standardizePath(await this.executeProcess(options)) as string
   }
 }