fixed mixed EOL in favor of cross platform

Author: Kelly Selden

__tests__/bin-test.js

@@ -145,7 +145,8 @@ describe('bin acceptance', function() {
           expect(stdout).toMatch('Done! Some files could not be upgraded automatically. See MODULE_REPORT.md.\n');
 
           let expectedOutput = fs.readFileSync(path.join(__dirname, 'expected', 'MODULE_REPORT.md'), 'utf8')
-            .replace(/\//, path.sep);
+            .replace(/\//, path.sep)
+            .replace(/\r?\n/g, require('os').EOL);
           let actualOutput = fs.readFileSync(path.join(tmpPath, 'MODULE_REPORT.md'), 'utf8');
 
           expect(actualOutput).toEqual(expectedOutput);

bin/ember-modules-codemod.js

@@ -99,7 +99,12 @@ function buildReport() {
         }
       });
 
-      fs.writeFileSync("MODULE_REPORT.md", "## Module Report\n" + report.join("\n"));
+      let file = "## Module Report\n" + report.join("\n");
+
+      // normalize line endings, so we don't end up with mixed
+      file = file.replace(/\r?\n/g, require('os').EOL);
+
+      fs.writeFileSync("MODULE_REPORT.md", file);
       console.log(chalk.yellow("\nDone! Some files could not be upgraded automatically. See " + chalk.blue("MODULE_REPORT.md") + "."));
     } else {
       console.log(chalk.green("\nDone! All uses of the Ember global have been updated."));