Skip to content
This repository was archived by the owner on Aug 7, 2021. It is now read-only.

revert: update script configuration #302

Merged
merged 2 commits into from
Oct 25, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 15 additions & 10 deletions bin/update-ns-webpack
Original file line number Diff line number Diff line change
@@ -1,16 +1,21 @@
#!/usr/bin/env node
const { resolve } = require("path");

const update = require("../verify/update");
const { getPackageJson, getProjectDir, writePackageJson } = require("../projectHelpers");
const { forceUpdateProjectDeps } = require("../dependencyManager");
const { editExistingProjectFiles } = require("../projectFilesManager");

const flags = JSON.parse(process.env.npm_config_argv).original
.filter(f => f.startsWith("--"))
.map(f => f.substring(2));
const PROJECT_DIR = getProjectDir({ nestingLvl: 2 });
const packageJson = getPackageJson(PROJECT_DIR);

function addOption(all, current) {
all[current] = true;
return all;
}
const options = flags.reduce(addOption, {});
console.info("Updating dev dependencies...");

update(options);
const { deps } = forceUpdateProjectDeps(packageJson);
packageJson.devDependencies = deps;
writePackageJson(packageJson, PROJECT_DIR);

console.info("\nUpdating configuration files...");
editExistingProjectFiles(PROJECT_DIR);

console.info("\nProject successfully updated! Don't forget to run `npm install`");

9 changes: 3 additions & 6 deletions installer.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,9 @@ function uninstall() {

projectFilesManager.removeProjectFiles(PROJECT_DIR, APP_DIR);

const scripts = packageJson.scripts;
if (scripts) {
console.log("Removing npm scripts...");
npmScriptsManager.removeDeprecatedNpmScripts(scripts);
npmScriptsManager.removeNpmScripts(scripts);
}
console.log("Removing npm scripts...");
npmScriptsManager.removeDeprecatedNpmScripts(packageJson);
npmScriptsManager.removeNpmScripts(packageJson.scripts);

helpers.writePackageJson(packageJson, PROJECT_DIR);

Expand Down
8 changes: 0 additions & 8 deletions npmScriptsManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,16 +53,8 @@ function removePlatformScripts(scripts, nameTemplate) {
});
}

function forceUpdateNpmScripts(scripts) {
removeDeprecatedNpmScripts(scripts);
removeNpmScripts(scripts);

addNpmScripts(scripts);
}

module.exports = {
addNpmScripts,
removeDeprecatedNpmScripts,
removeNpmScripts,
forceUpdateNpmScripts,
};
9 changes: 1 addition & 8 deletions projectFilesManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,15 +59,10 @@ function removeProjectFiles(projectDir, appDir) {
});
}

function forceUpdateProjectFiles(projectDir, appDir) {
removeProjectFiles(projectDir, appDir);
addProjectFiles(projectDir, appDir);
}

function deleteFile(destinationPath) {
if (fs.existsSync(destinationPath)) {
console.info(`Deleting file: ${destinationPath}`);
fs.unlinkSync(destinationPath);
fs.unlink(destinationPath);
}
}

Expand Down Expand Up @@ -194,7 +189,5 @@ function tsOrJs(projectDir, name) {
module.exports = {
addProjectFiles,
removeProjectFiles,
forceUpdateProjectFiles,
editExistingProjectFiles,
};

106 changes: 0 additions & 106 deletions verify/update.js

This file was deleted.