fix: update workflow inputs to use correct types and remove unnecessary environment variable

Author: Le-Caignec

.github/workflows/deploy.yml

@@ -17,6 +17,13 @@ on:
 
 jobs:
 
+  # Build and test before deploying.
+  ci:
+    uses: ./.github/workflows/main.yml
+    with:
+      SEPOLIA_RPC_URL: ${{ vars.SEPOLIA_RPC_URL }}
+      ARBITRUM_SEPOLIA_RPC_URL: ${{ vars.ARBITRUM_SEPOLIA_RPC_URL }}
+
   # Build and test before deploying.
   ci:
     uses: ./.github/workflows/main.yml

.github/workflows/main.yml

@@ -3,11 +3,13 @@ name: Default
 on:
   pull_request:
   workflow_call:
-    secrets:
+    inputs:
       SEPOLIA_RPC_URL:
         required: true
+        type: string
       ARBITRUM_SEPOLIA_RPC_URL:
         required: true
+        type: string
 
 concurrency:
   group: ${{ github.ref }}-ci
@@ -20,7 +22,6 @@ jobs:
   build-and-test:
     name: Build and test
     runs-on: ubuntu-latest
-    environment: 'testnets'
     steps:
       - uses: actions/checkout@v4
         with:
@@ -44,8 +45,8 @@ jobs:
       - name: Run Foundry coverage
         env:
           CI: true
-          SEPOLIA_RPC_URL: ${{ secrets.SEPOLIA_RPC_URL }}
-          ARBITRUM_SEPOLIA_RPC_URL: ${{ secrets.ARBITRUM_SEPOLIA_RPC_URL }}
+          SEPOLIA_RPC_URL: ${{ inputs.SEPOLIA_RPC_URL || vars.SEPOLIA_RPC_URL }}
+          ARBITRUM_SEPOLIA_RPC_URL: ${{ inputs.ARBITRUM_SEPOLIA_RPC_URL || vars.ARBITRUM_SEPOLIA_RPC_URL }}
         run: make generate-coverage
 
       - name: Upload coverage reports to Codecov