Add deprecation notices to legacy commands (#1227)

* Add deprecation notices to legacy commands.

bin/legacy/commands/ci.js

@@ -1,6 +1,10 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/repo/prepare');
 
 exports.command = 'ci';
 exports.description = 'Prepare Git repository for CML operations';
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice: '"cml ci" is deprecated, please use "cml repo prepare"'
+});

bin/legacy/commands/publish.js

@@ -1,6 +1,10 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/asset/publish');
 
 exports.command = 'publish <asset>';
 exports.description = false;
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice: '"cml publish" is deprecated, please use "cml asset publish"'
+});

bin/legacy/commands/rerun-workflow.js

@@ -1,6 +1,10 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/workflow/rerun');
 
 exports.command = 'rerun-workflow';
 exports.description = false;
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice: '"cml rerun-workflow" is deprecated, please use "cml workflow rerun"'
+});

bin/legacy/commands/send-comment.js

@@ -1,6 +1,10 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/comment/create');
 
 exports.command = 'send-comment <markdown file>';
 exports.description = false;
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice: '"cml send-comment" is deprecated, please use "cml comment create"'
+});

bin/legacy/commands/send-github-check.js

@@ -1,6 +1,10 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/check/create');
 
 exports.command = 'send-github-check <markdown file>';
 exports.description = false;
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice: '"cml send-github-check" is deprecated, please use "cml check create"'
+});

bin/legacy/commands/tensorboard-dev.js

@@ -1,6 +1,11 @@
+const { addDeprecationNotice } = require('../deprecation');
 const { builder, handler } = require('../../cml/tensorboard/connect');
 
 exports.command = 'tensorboard-dev';
 exports.description = false;
 exports.handler = handler;
-exports.builder = builder;
+exports.builder = addDeprecationNotice({
+  builder,
+  notice:
+    '"cml tensorboard-dev" is deprecated, please use "cml tensorboard connect"'
+});

bin/legacy/deprecation.js

@@ -0,0 +1,20 @@
+const winston = require('winston');
+
+// addDeprecationNotice adds middleware to the yargs chain to display a deprecation notice.
+const addDeprecationNotice = (opts = {}) => {
+  const { builder, notice } = opts;
+  return (yargs) =>
+    builder(yargs).middleware([(opts) => deprecationNotice(opts, notice)]);
+};
+
+const deprecationNotice = (opts, notice) => {
+  const { cml } = opts;
+  const driver = cml.getDriver();
+  if (driver.warn) {
+    driver.warn(notice);
+  } else {
+    winston.warn(notice);
+  }
+};
+
+exports.addDeprecationNotice = addDeprecationNotice;

src/drivers/github.js

@@ -701,6 +701,10 @@ class Github {
     return command;
   }
 
+  warn(message) {
+    console.error(`::warning::${message}`);
+  }
+
   get sha() {
     if (GITHUB_EVENT_NAME === 'pull_request')
       return github.context.payload.pull_request.head.sha;