fix(core): output paths still use windows separator on linux / mac

nx-dotnet · Apr 26, 2021 · b729fed · b729fed
1 parent fa8f507
commit b729fed
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/.releaserc.js b/.releaserc.js
@@ -37,7 +37,7 @@ module.exports = {
       {
         publishCmd: [
           'yarn ts-node tools/scripts/publish-all ${nextRelease.version} ${nextRelease.channel}',
-        ],
+        ].join(' && '),
       },
     ],
     [

diff --git a/packages/core/src/generators/utils/generate-project.spec.ts b/packages/core/src/generators/utils/generate-project.spec.ts
@@ -72,7 +72,7 @@ describe('nx-dotnet project generator', () => {
     expect(outputPath).toBeTruthy();
 
     // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-    const absoluteDistPath = resolve(resolve(config.root, outputPath));
+    const absoluteDistPath = resolve(config.root, outputPath);
     const expectedDistPath = resolve('./dist/test');
 
     expect(absoluteDistPath).toEqual(expectedDistPath);

diff --git a/packages/core/src/generators/utils/generate-project.ts b/packages/core/src/generators/utils/generate-project.ts
@@ -138,11 +138,14 @@ function SetOutputPath(
     readFileSync(projectFilePath).toString()
   );
 
+  let outputPath = `${relative(
+    dirname(projectFilePath),
+    process.cwd()
+  )}/dist/${projectName}`;
+  outputPath = outputPath.replace('\\', '/'); // Forward slash works on windows, backslash does not work on mac/linux
+
   const textNode: Partial<XmlTextNode> = {
-    text: `${relative(
-      dirname(projectFilePath),
-      process.cwd()
-    )}\\dist\\${projectName}`,
+    text: outputPath,
     type: 'text',
   };
   textNode.toString = () => textNode.text ?? '';
-Original file line number
+Diff line change
@@ Expand Up / @@ -37,7 +37,7 @@ module.exports = { @@
           {
             publishCmd: [
               'yarn ts-node tools/scripts/publish-all ${nextRelease.version} ${nextRelease.channel}',
-            ],
+            ].join(' && '),
           },
         ],
         [
@@ Expand Down @@