Remove deprecated web api methods

src/WebClient.spec.js

@@ -919,23 +919,6 @@ describe('WebClient', function () {
       this.capture.startCapture();
     });
 
-    it('should warn when calling a deprecated method', function () {
-      return this.client.files.comments.add({ file: 'FILE', comment: 'COMMENT' })
-        .then(() => {
-          const output = this.capture.getCapturedText();
-          assert.isNotEmpty(output);
-          const warning = output[0];
-          assert.match(warning, /^\[WARN\]/);
-          this.scope.done();
-        })
-        .then(() => {
-          this.capture.stopCapture();
-        }, (error) => {
-          this.capture.stopCapture();
-          throw error;
-        });
-    });
-
     it('should warn when calling an API method using a callback', function (done) {
       this.client.apiCall('method', {}, () => {
         const output = this.capture.getCapturedText();

src/WebClient.ts

@@ -146,15 +146,6 @@ export class WebClient extends EventEmitter {
         throw new TypeError(`Expected an options argument but instead received a ${typeof options}`);
       }
 
-      // warn for methods whose functionality is deprecated
-      if (method === 'files.comments.add' || method === 'files.comments.edit') {
-        this.logger.warn(
-          `File comments are deprecated in favor of file threads. Replace uses of ${method} in your app ` +
-          'to take advantage of improvements. See https://api.slack.com/changelog/2018-05-file-threads-soon-tread ' +
-          'to learn more.',
-        );
-      }
-
       const methodSupportsCursorPagination = methods.cursorPaginationEnabledMethods.has(method);
       const optionsPaginationType = getOptionsPaginationType(options);
 
@@ -378,9 +369,7 @@ export class WebClient extends EventEmitter {
       (this.apiCall.bind(this, 'files.sharedPublicURL')) as Method<methods.FilesSharedPublicURLArguments>,
     upload: (this.apiCall.bind(this, 'files.upload')) as Method<methods.FilesUploadArguments>,
     comments: {
-      add: (this.apiCall.bind(this, 'files.comments.add')) as Method<methods.FilesCommentsAddArguments>,
       delete: (this.apiCall.bind(this, 'files.comments.delete')) as Method<methods.FilesCommentsDeleteArguments>,
-      edit: (this.apiCall.bind(this, 'files.comments.edit')) as Method<methods.FilesCommentsEditArguments>,
     },
   };
 

src/methods.ts

@@ -612,19 +612,10 @@ export type FilesUploadArguments = TokenOverridable & {
   title?: string;
   thread_ts?: string; // if specified, `channels` must be set
 };
-export type FilesCommentsAddArguments = TokenOverridable & {
-  comment: string;
-  file: string; // file id
-};
 export type FilesCommentsDeleteArguments = TokenOverridable & {
   file: string; // file id
   id: string; // comment id
 };
-export type FilesCommentsEditArguments = TokenOverridable & {
-  comment: string;
-  file: string; // file id
-  id: string; // comment id
-};
 
   /*
    * `groups.*`