]> git.wincent.com - wincent.git/commitdiff
refactor(fig): improve consistency of logging
authorGreg Hurrell <greg@hurrell.net>
Thu, 2 Apr 2020 23:14:25 +0000 (01:14 +0200)
committerGreg Hurrell <greg@hurrell.net>
Thu, 2 Apr 2020 23:14:25 +0000 (01:14 +0200)
src/Fig/operations/file.ts
src/Fig/operations/template.ts
src/Fig/status.ts
src/main.ts

index 424c45ef7c593ae06f63321264876479d90a905f..6bd2ec1f363eab2bcac7eb9353c490d40e08210b 100644 (file)
@@ -39,7 +39,7 @@ function directory(path: string) {
         log.error(`${path} already exists but is not a directory`);
       }
     } catch (error) {
-      log.error(`failed to stat: ${path}`);
+      log.error(`Failed to stat: ${path}`);
     }
   } else {
     Context.informChanged(`directory ${path}`);
index 04c81d544706852f8aae0ac3f0a645c2004ee565..2f81d6ae246fa9d6941a4a5cc39ad43a029d4ba2 100644 (file)
@@ -41,7 +41,7 @@ export default async function template({
   });
 
   if (owner && owner !== Context.attributes.username) {
-    log.notice(`needs sudo: ${Context.attributes.username} -> ${owner}`);
+    log.debug(`Needs sudo: ${Context.attributes.username} -> ${owner}`);
     const passphrase = await Context.sudoPassphrase;
     const result = await run('ls', ['-l', '/var/audit'], {passphrase});
 
@@ -53,7 +53,7 @@ export default async function template({
     }
   } else {
     if (diff.contents) {
-      log.info('change!');
+      // log.info('change!');
       const temp = await tempfile(contents);
 
       // TODO: cp from temp to target
index 3c5ac1a0f2b2feeb444047f50ef28c8a24e850f7..fbe728b597a81363bd0801ab1ec35be3024eaed6 100644 (file)
@@ -4,17 +4,17 @@ import {log} from '../console';
 // decide whether we need any of those.
 
 export function changed(message: string) {
-  log.notice(`changed: ${message}`);
+  log.notice(`Changed: ${message}`);
 }
 
 export function failed(message: string) {
-  log.error(`failed: ${message}`);
+  log.error(`Failed: ${message}`);
 }
 
 export function ok(message: string) {
-  log.info(`ok: ${message}`);
+  log.info(`Ok: ${message}`);
 }
 
 export function skipped(message: string) {
-  log.info(`skipped: ${message}`);
+  log.info(`Skipped: ${message}`);
 }
index 1d9e4fd42a8fc6ca9c5bbbd36ac6be379b1a95d0..be245771586dde4eff6e4e63d499a6969ec08fd3 100644 (file)
@@ -169,7 +169,7 @@ async function main() {
       for (const [callback, name] of Context.tasks.get(aspect)) {
         if (!options.startAt.found || name === options.startAt.literal) {
           options.startAt.found = false;
-          log.info(`task: ${name}`);
+          log.info(`Task: ${name}`);
 
           await Context.withContext({aspect, variables}, async () => {
             await callback();