addressing feedback: add workflows and fix comments

.github/workflows/coverage.yml

@@ -0,0 +1,49 @@
+name: Code Coverage
+
+on:
+  push:
+  pull_request:
+  merge_group:
+
+jobs:
+  coverage:
+    name: coverage
+    runs-on: ubuntu-latest-16core
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Install Foundry
+        uses: foundry-rs/foundry-toolchain@v1
+        with:
+          version: nightly
+          cache: false
+
+      - name: forge version
+        run: forge --version
+
+      - name: Run coverage
+        run: |
+          make coverage
+          sudo apt-get install lcov
+          lcov --remove lcov.info  -o lcov.info 'test/*' 'script/*'
+
+      - name: Edit lcov.info
+        run: |
+          cat lcov.info | sed '/.*\/test\/.*/,/TN:/d' > tmp.info && mv tmp.info lcov.info
+
+      - name: Coveralls
+        uses: coverallsapp/github-action@v2
+        with:
+          path-to-lcov: "./lcov.info"
+          parallel: true
+
+  finish:
+    needs: coverage
+    runs-on: ubuntu-latest
+    steps:
+      - name: Coveralls Finished
+        uses: coverallsapp/github-action@v2
+        with:
+          parallel-finished: true

.github/workflows/test.yml

@@ -1,6 +1,9 @@
 name: test
 
-on: workflow_dispatch
+on:
+  push:
+  pull_request:
+  merge_group:
 
 env:
   FOUNDRY_PROFILE: ci
@@ -24,11 +27,10 @@ jobs:
 
       - name: Run Forge build
         run: |
-          forge --version
-          forge build --sizes
+          make build
         id: build
 
       - name: Run Forge tests
         run: |
-          forge test -vvv
+          make test
         id: test

Makefile

@@ -0,0 +1,14 @@
+.PHONY: build test
+
+### Build ###
+
+build:
+	forge build --force --sizes
+
+### Test ###
+
+test:
+	forge test
+
+coverage:
+	forge coverage --report lcov

contracts/internal/EverlongBase.sol

@@ -8,6 +8,8 @@ import { IEverlongEvents } from "../interfaces/IEverlongEvents.sol";
 import { EVERLONG_KIND, EVERLONG_VERSION } from "../libraries/Constants.sol";
 import { EverlongERC4626 } from "./EverlongERC4626.sol";
 
+// TODO: Reassess whether centralized configuration management makes sense.
+//       https://github.com/delvtech/everlong/pull/2#discussion_r1703799747
 /// @author DELV
 /// @title EverlongBase
 /// @notice Base contract for Everlong.
@@ -21,12 +23,12 @@ abstract contract EverlongBase is EverlongERC4626, IEverlongEvents {
     // │ Storage                                                 │
     // ╰─────────────────────────────────────────────────────────╯
 
-    /// Admin ///
+    // Admin //
 
     /// @dev Address of the contract admin.
     address internal _admin;
 
-    /// Hyperdrive ///
+    // Hyperdrive //
 
     /// @dev Address of the Hyperdrive instance wrapped by Everlong.
     address public immutable hyperdrive;
@@ -35,7 +37,7 @@ abstract contract EverlongBase is EverlongERC4626, IEverlongEvents {
     //          If false, use the Hyperdrive's `vaultSharesToken`.
     bool internal immutable _asBase;
 
-    /// Positions ///
+    // Positions //
 
     // TODO: Reassess using a more tailored data structure.
     /// @dev Utility data structure to manage the position queue.