Follows style guide

Author: nikita-skobov

server/src/__tests__/analyzer.test.ts

@@ -201,7 +201,7 @@ describe('commentsAbove', () => {
   it('handles line breaks in comments', () => {
     analyzer.analyze(CURRENT_URI, FIXTURES.COMMENT_DOC)
     expect(analyzer.commentsAbove(CURRENT_URI, 28)).toEqual(
-        'doc for func_two\nhas two lines',
+      'doc for func_two\nhas two lines',
     )
   })
 
@@ -239,8 +239,7 @@ describe('fromRoot', () => {
 
     expect(connection.window.showWarningMessage).not.toHaveBeenCalled()
 
-    // TODO: maybe not hardcode this number in case
-    // fixture files are added in the future?
+    // if you add a .sh file to testing/fixtures, update this value
     const FIXTURE_FILES_MATCHING_GLOB = 11
 
     // Intro, stats on glob, one file skipped due to shebang, and outro

server/src/analyser.ts

@@ -415,7 +415,7 @@ export default class Analyzer {
     // is not a comment line
     const getComment = (l: string): null | string => {
       // this regexp has to be defined within the function
-      const commentRegExp = /^\s*\#\s*(.*)/g
+      const commentRegExp = /^\s*#\s*(.*)/g
       const matches = commentRegExp.exec(l)
       return matches ? matches[1].trim() : null
     }
@@ -428,7 +428,7 @@ export default class Analyzer {
     // iterate on every line above and including
     // the current line until getComment returns null
     let currentComment: string | null = ''
-    while (currentComment = getComment(currentLine)) {
+    while ((currentComment = getComment(currentLine))) {
       commentBlock.push(currentComment)
       commentBlockIndex -= 1
       currentLine = doc.getText({