revert stack run button, and remove some debug code

Author: Meowcolm024

CHANGELOG.md

@@ -6,10 +6,9 @@ All notable changes to the *Haskell Runner 2* will be documented in this file.
 
 - nil
 
-## [0.4.0] (Ureleased)
+## [0.4.0]
 
-- avoid reusing recovered terminal at startup by keeping track of active terminals created by Haskell Runner 2
-- `Stack Run` button can now be hot loaded, but disabling still requires reload
+- avoid reusing recovered terminal at startup by keeping track of active terminals created the extension
 - remove activate on `Cabal` language
 
 ## [0.3.0]

README.md

@@ -1,6 +1,6 @@
 # Haskell Runner 2
 
-Successor of the original `Haskell Runner`.
+Rewrite of the original `Haskell Runner`.
 
 ## Features
 
@@ -22,7 +22,7 @@ The following configurations are available:
 - `runner2.stackPath`: path for stack executable (default `stack`)
 - `runner2.cabalPath`: path for cabal executable (default `cabal`)
 - `runner2.replTool`: by `default`, it use `ghci` in single file and use `stack repl` (or `cabal repl`) in stack project. This can be overridden by choosing other options (`ghci` for ghci only, and `stack` (or `cabal`) for stack/cabal repl only)
-- `runner2.showRunButton`: show "Stack/Cabal Run" button (default `false`)
+- `runner2.showRunButton`: show "Stack/Cabal Run" button, *reload required* (default `false`)
 
 ## Release Notes
 

src/extension.ts

@@ -9,29 +9,18 @@ import * as util from './util';
 var config: conf.Config = conf.getConfig();
 // map of saved terminals of current session
 var terminal: Map<string, vscode.Terminal> = new Map();
-// stack run button already initialized
-var stackRunBtn: boolean = false;
 
 // I'm not sure if we can do `async` here
 export async function activate(context: vscode.ExtensionContext) {
     // check stack/cabal project
     const hasConf = async (f: string) => (await vscode.workspace.findFiles(f)).length > 0;
     let project: conf.ProjectTy = (await hasConf("stack.yaml")) ? "stack" : (await hasConf("*.cabal") ? "cabal" : "none");
     let inproject = project !== "none";
+
     // update config
+    // ? I'm not sure if it is useful or not
     vscode.workspace.onDidChangeConfiguration(e => {
         config = conf.getConfig();
-        if (config.showRun && !stackRunBtn) {
-            switch (project) {
-                case 'stack':
-                    stackRunBtn = true;
-                    return util.resgisterStatButton(context, "Stack Run", "runner2.hsrun");
-                case 'cabal':
-                    stackRunBtn = true;
-                    return util.resgisterStatButton(context, "Cabal Run", "runner2.hsrun");
-                default: return;
-            }
-        }
     });
 
     // GHCi command
@@ -73,7 +62,7 @@ export async function activate(context: vscode.ExtensionContext) {
             .map(s => ":{\n" + s + "\n:}\n")            // in case of multi-line selection
             .map(s => {
                 const term = util.getTermOption(terminal, "GHCi")
-                    .map(term => () => { console.log(term.exitStatus, term.state); return term; })
+                    .map(term => () => term)
                     .orelse(() => {
                         let term = vscode.window.createTerminal("GHCi");
                         term.sendText(config.ghciTool(project));     // we're not loading the file here
@@ -98,7 +87,6 @@ export async function activate(context: vscode.ExtensionContext) {
         util.resgisterStatButton(context, project + " Build", "runner2.hsbuild");
         util.resgisterStatButton(context, project + " Test", "runner2.hstest");
         if (config.showRun) {
-            stackRunBtn = true;
             util.resgisterStatButton(context, project + " Run", "runner2.hsrun");
         }
     };

src/util.ts

@@ -44,7 +44,6 @@ export function registerSimplTerm(
             })();
         term.sendText(cmd);
         term.show();
-
     }));
 }