Internal review

.github/workflows/test.yml

@@ -1,34 +1,73 @@
 name: test
 
-on: workflow_dispatch
-
-env:
-  FOUNDRY_PROFILE: ci
+on:
+  push:
+      branches:
+          - master
+  pull_request:
 
 jobs:
-  check:
-    strategy:
-      fail-fast: true
-
-    name: Foundry project
+  build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-        with:
-          submodules: recursive
-
-      - name: Install Foundry
-        uses: foundry-rs/foundry-toolchain@v1
-        with:
-          version: nightly
-
-      - name: Run Forge build
-        run: |
-          forge --version
-          forge build --sizes
-        id: build
-
-      - name: Run Forge tests
-        run: |
-          forge test -vvv
-        id: test
+        - uses: actions/checkout@v3
+          with:
+              submodules: recursive
+
+        - name: Install foundry
+          uses: foundry-rs/foundry-toolchain@v1
+          with:
+              version: nightly
+
+        - name: Run foundry build
+          run: |
+              forge --version
+              forge build
+          id: build
+
+  lint-check:
+      runs-on: ubuntu-latest
+      needs: build
+      steps:
+          - uses: actions/checkout@v3
+            with:
+                submodules: recursive
+          - uses: foundry-rs/foundry-toolchain@v1
+            with:
+                version: nightly
+
+          - name: Run foundry fmt check
+            run: |
+                forge fmt --check
+            id: fmt
+
+  test:
+      runs-on: ubuntu-latest
+      needs: lint-check
+      steps:
+          - uses: actions/checkout@v3
+            with:
+                submodules: recursive
+          - uses: foundry-rs/foundry-toolchain@v1
+            with:
+                version: nightly
+          - name: Run foundry tests
+            # --ast tests enables inline configs to work https://github.com/foundry-rs/foundry/issues/7310#issuecomment-1978088200
+            run: |
+                forge test -vvv --gas-report --ast 
+            id: test
+
+  coverage:
+      runs-on: ubuntu-latest
+      needs: lint-check
+      steps:
+          - uses: actions/checkout@v3
+            with:
+                submodules: recursive
+          - uses: foundry-rs/foundry-toolchain@v1
+            with:
+                version: nightly
+          - name: Run foundry coverage
+            run: |
+                FOUNDRY_PROFILE=coverage forge coverage --report summary
+            id: coverage

.gitignore

@@ -1,3 +1,16 @@
+# IDE - VSCode
+.vscode/
+
+# Logs
+logs/
+*.log
+
+# Dependency directories
+node_modules/
+
+# Optional npm cache directory
+.npm/
+
 # Compiler files
 cache/
 out/
@@ -7,8 +20,16 @@ out/
 /broadcast/*/31337/
 /broadcast/**/dry-run/
 
-# Docs
-docs/
-
 # Dotenv file
-.env
+*.env
+
+# System Files
+.DS_Store
+Thumbs.db
+
+# Coverage
+coverage/
+*.info
+
+# Gas snapshot
+.gas-snapshot

.gitmodules

@@ -0,0 +1,9 @@
+[submodule "lib/forge-std"]
+	path = lib/forge-std
+	url = https://github.com/foundry-rs/forge-std
+[submodule "lib/openzeppelin-contracts"]
+	path = lib/openzeppelin-contracts
+	url = https://github.com/openzeppelin/openzeppelin-contracts
+[submodule "lib/ethereum-vault-connector"]
+	path = lib/ethereum-vault-connector
+	url = https://github.com/euler-xyz/ethereum-vault-connector