Update comments and remove no-any rule

src/apis.ts

@@ -19,6 +19,7 @@ export enum SCRIPT_TYPES {
 
 // Also see:
 // https://github.com/SchemaStore/schemastore/blob/master/src/schemas/json/appsscript.json
+// TODO: unnecessary export
 export interface AdvancedService {
   userSymbol: string;
   serviceId: string;

src/apiutils.ts

@@ -47,6 +47,7 @@ export async function getFunctionNames(script: script_v1.Script, scriptId: strin
 /**
  * Gets the project ID from the manifest. If there is no project ID, it returns an error.
  */
+// TODO: unnecessary export
 export async function getProjectIdWithErrors() {
   const projectId = await getProjectId(); // will prompt user to set up if required
   if (!projectId) throw logError(null, ERROR.NO_GCLOUD_PROJECT);

src/auth.ts

@@ -310,6 +310,7 @@ async function setOauthClientCredentials(rc: ClaspToken) {
  * authorize if new scopes found (local OAuth credentails only).
  * @param {ClaspToken} rc OAuth client settings from rc file.
  */
+// TODO: currently unused. Check relevancy
 export async function checkOauthScopes(rc: ClaspToken) {
   try {
     await checkIfOnline();

src/commands/logs.ts

@@ -1,4 +1,5 @@
 import chalk from 'chalk';
+import { GaxiosResponse } from 'gaxios';
 import { logging_v2 } from 'googleapis';
 import * as open from 'open';
 import { loadAPICredentials, logger } from './../auth';
@@ -59,6 +60,7 @@ export default async (cmd: { json: boolean; open: boolean; setup: boolean; watch
  * Prints log entries
  * @param entries {any[]} StackDriver log entries.
  */
+// TODO: unnecessary export
 export function printLogs(entries: logging_v2.Schema$LogEntry[] = [], formatJson: boolean) {
   /**
    * This object holds all log IDs that have been printed to the user.
@@ -104,8 +106,7 @@ export function printLogs(entries: logging_v2.Schema$LogEntry[] = [], formatJson
         payloadData = padEnd(payloadData, 20);
       }
     }
-    // tslint:disable-next-line:no-any
-    const coloredStringMap: any = {
+    const coloredStringMap: { [key: string]: string } = {
       ERROR: chalk.red(severity),
       INFO: chalk.cyan(severity),
       DEBUG: chalk.green(severity), // includes timeEnd
@@ -122,6 +123,7 @@ export function printLogs(entries: logging_v2.Schema$LogEntry[] = [], formatJson
   }
 }
 
+// TODO: unnecessary export
 export async function setupLogs(): Promise<string> {
   let projectId: string;
   const promise = new Promise<string>((resolve, reject) => {
@@ -161,6 +163,7 @@ export async function setupLogs(): Promise<string> {
  * Fetches the logs and prints the to the user.
  * @param startDate {Date?} Get logs from this date to now.
  */
+// TODO: unnecessary export
 export async function fetchAndPrintLogs(formatJson: boolean, projectId?: string, startDate?: Date) {
   const oauthSettings = await loadAPICredentials();
   spinner.setSpinnerTitle(`${oauthSettings.isLocalCreds ? LOG.LOCAL_CREDS : ''}${LOG.GRAB_LOGS}`).start();
@@ -192,8 +195,7 @@ export async function fetchAndPrintLogs(formatJson: boolean, projectId?: string,
       console.log(filter);
     }
     // Parse response and print logs or print error message.
-    // tslint:disable-next-line:no-any
-    const parseResponse = (response: any) => {
+    const parseResponse = (response: GaxiosResponse<logging_v2.Schema$ListLogEntriesResponse>) => {
       if (logs.status !== 200) {
         switch (logs.status) {
           case 401:

src/commands/push.ts

@@ -33,6 +33,7 @@ export default async (cmd: { watch: boolean; force: boolean }) => {
     /**
      * @see https://www.npmjs.com/package/watch
      */
+    // TODO check alternative https://github.com/paulmillr/chokidar
     watchTree(rootDir || '.', async (f, curr, prev) => {
       // The first watch doesn't give a string for some reason.
       if (typeof f === 'string') {

src/files.ts

@@ -52,6 +52,7 @@ export function hasProject(): boolean {
  * Returns in tsconfig.json.
  * @returns {ts.TranspileOptions} if tsconfig.json not exists, return undefined.
  */
+// TODO: unnecessary export
 export function getTranspileOptions(): ts.TranspileOptions {
   const projectPath = findUp.sync(DOT.PROJECT.PATH);
   const tsconfigPath = path.join(projectPath ? path.dirname(projectPath) : DOT.PROJECT.DIR, 'tsconfig.json');

src/index.ts

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+
 /**
  * @license
  * Copyright Google Inc.

src/manifest.ts

@@ -33,6 +33,7 @@ export async function readManifest(): Promise<Manifest> {
  * Writes the appsscript.json manifest file.
  * @param {Manifest} manifest The new manifest to write.
  */
+// TODO: unnecessary export
 export async function writeManifest(manifest: Manifest) {
   let { rootDir } = await getProjectSettings();
   if (typeof rootDir === 'undefined') rootDir = DOT.PROJECT.DIR;
@@ -99,6 +100,7 @@ export async function addScopeToManifest(scopes: string[]) {
  * Enables the Execution API in the Manifest.
  * The Execution API requires the manifest to have the "executionApi.access" field set.
  */
+// TODO: currently unused. Check relevancy
 export async function enableExecutionAPI() {
   console.log('Writing manifest');
   const manifest = await readManifest();
@@ -344,6 +346,7 @@ interface Gmail {
   sheets: Sheets;
 }
 
+// TODO: unnecessary export
 export interface Manifest {
   timeZone?: string;
   oauthScopes?: string[];

src/utils.ts

@@ -46,7 +46,7 @@ export const hasOauthClientSettings = (local = false): boolean =>
 /**
  * Gets the OAuth client settings from rc file.
  * @param {boolean} local If true, gets the local OAuth settings. Global otherwise.
- * Should be used instead of `DOTFILE.RC?().read()`
+ * ! Should be used instead of `DOTFILE.RC?().read()`
  * @returns {Promise<ClaspToken>} A promise to get the rc file as object.
  */
 export function getOAuthSettings(local: boolean): Promise<ClaspToken> {
@@ -264,7 +264,7 @@ export function getDefaultProjectName(): string {
 
 /**
  * Gets the project settings from the project dotfile. Logs errors.
- * Should be used instead of `DOTFILE.PROJECT().read()`
+ * ! Should be used instead of `DOTFILE.PROJECT().read()`
  * @param  {boolean} failSilently Don't err when dot file DNE.
  * @return {Promise<ProjectSettings>} A promise to get the project dotfile as object.
  */
@@ -335,6 +335,7 @@ export async function saveProject(
   append = true): Promise<ProjectSettings> {
   if (append) {
     const projectSettings: ProjectSettings = await DOTFILE.PROJECT().read();
+    // TODO: const projectSettings: ProjectSettings = await getProjectSettings();
     newProjectSettings = { ...projectSettings, ...newProjectSettings };
   }
   return DOTFILE.PROJECT().write(newProjectSettings);