feat: use common logger part 1

Introduce logger to bundle core and eject

packages/cli/src/bundle/buildBundle.js

@@ -14,10 +14,9 @@ const Server = require('metro/src/Server');
 
 const outputBundle = require('metro/src/shared/output/bundle');
 const path = require('path');
-const log = require('./log').out('bundle');
 const saveAssets = require('./saveAssets');
-
 const loadMetroConfig = require('../util/loadMetroConfig');
+const logger = require('../util/logger');
 
 async function buildBundle(
   args: CommandLineArgs,
@@ -51,7 +50,7 @@ async function buildBundle(
   try {
     const bundle = await output.build(server, requestOpts);
 
-    await output.save(bundle, args, log);
+    await output.save(bundle, args, logger.info);
 
     // Save the assets of the bundle
     const outputAssets = await server.getAssets({

packages/cli/src/bundle/bundleCommandLineArgs.js

@@ -96,11 +96,6 @@ module.exports = [
     description:
       'Directory name where to store assets referenced in the bundle',
   },
-  {
-    command: '--verbose',
-    description: 'Enables logging',
-    default: false,
-  },
   {
     command: '--reset-cache',
     description: 'Removes cached files',

packages/cli/src/bundle/saveAssets.js

@@ -14,11 +14,11 @@ const fs = require('fs');
 const filterPlatformAssetScales = require('./filterPlatformAssetScales');
 const getAssetDestPathAndroid = require('./getAssetDestPathAndroid');
 const getAssetDestPathIOS = require('./getAssetDestPathIOS');
-const log = require('./log').out('bundle');
+const logger = require('../util/logger');
 
 function saveAssets(assets, platform, assetsDest) {
   if (!assetsDest) {
-    console.warn('Assets destination folder is not set, skipping...');
+    logger.warn('Assets destination folder is not set, skipping...');
     return Promise.resolve();
   }
 
@@ -49,15 +49,15 @@ function copyAll(filesToCopy) {
     return Promise.resolve();
   }
 
-  log(`Copying ${queue.length} asset files`);
+  logger.info(`Copying ${queue.length} asset files`);
   return new Promise((resolve, reject) => {
     const copyNext = error => {
       if (error) {
         reject(error);
         return;
       }
       if (queue.length === 0) {
-        log('Done copying assets');
+        logger.info('Done copying assets');
         resolve();
       } else {
         const src = queue.shift();

packages/cli/src/core/getCommands.js

@@ -6,6 +6,7 @@ import type { CommandT, ProjectCommandT, LocalCommandT } from './types.flow';
 
 const path = require('path');
 const findPlugins = require('./findPlugins');
+const logger = require('../util/logger');
 
 /**
  * List of built-in commands
@@ -84,7 +85,7 @@ module.exports = (root: string): Array<CommandT> => [
   {
     name: 'init',
     func: () => {
-      console.log(
+      logger.warn(
         [
           'Looks like a React Native project already exists in the current',
           'folder. Run this command from a different folder or remove node_modules/react-native',

packages/cli/src/eject/eject.js

@@ -10,6 +10,7 @@
 const path = require('path');
 const fs = require('fs');
 const copyProjectTemplateAndReplace = require('../generator/copyProjectTemplateAndReplace');
+const logger = require('../util/logger');
 
 /**
  * The eject command re-creates the `android` and `ios` native folders. Because native code can be
@@ -26,7 +27,7 @@ function eject() {
   const doesIOSExist = fs.existsSync(path.resolve('ios'));
   const doesAndroidExist = fs.existsSync(path.resolve('android'));
   if (doesIOSExist && doesAndroidExist) {
-    console.error(
+    logger.error(
       'Both the iOS and Android folders already exist! Please delete `ios` and/or `android` ' +
         'before ejecting.'
     );
@@ -37,7 +38,7 @@ function eject() {
   try {
     appConfig = require(path.resolve('app.json'));
   } catch (e) {
-    console.error(
+    logger.error(
       'Eject requires an `app.json` config file to be located at ' +
         `${path.resolve(
           'app.json'
@@ -49,7 +50,7 @@ function eject() {
 
   const appName = appConfig.name;
   if (!appName) {
-    console.error(
+    logger.error(
       'App `name` must be defined in the `app.json` config file to define the project name. ' +
         'It must not contain any spaces or dashes.'
     );
@@ -59,7 +60,7 @@ function eject() {
   // eslint-disable-next-line prefer-destructuring
   const displayName = appConfig.displayName;
   if (!displayName) {
-    console.error(
+    logger.error(
       'App `displayName` must be defined in the `app.json` config file, to define the label ' +
         'of the app on the home screen.'
     );
@@ -69,7 +70,7 @@ function eject() {
   const templateOptions = { displayName };
 
   if (!doesIOSExist) {
-    console.log('Generating the iOS folder.');
+    logger.info('Generating the iOS folder.');
     copyProjectTemplateAndReplace(
       path.resolve(
         'node_modules',
@@ -87,7 +88,7 @@ function eject() {
   }
 
   if (!doesAndroidExist) {
-    console.log('Generating the Android folder.');
+    logger.info('Generating the Android folder.');
     copyProjectTemplateAndReplace(
       path.resolve(
         'node_modules',