Skip to content

Commit

Permalink
Merge pull request #246 from FrancoAguzzi/fix/semantic-release-config
Browse files Browse the repository at this point in the history
fix: add .env vars to release github action
  • Loading branch information
FrancoAguzzi authored Mar 19, 2024
2 parents 58ccad4 + 7bcd434 commit 0d996e9
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,12 @@ jobs:
- run: pnpm audit signatures
- name: Release
env:
NEXT_PUBLIC_WALLET_CONNECT_PROJECT_ID: ${{ secrets.NEXT_PUBLIC_WALLET_CONNECT_PROJECT_ID }}
NEXT_PUBLIC_ALCHEMY_ETHEREUM_HTTP: ${{ secrets.NEXT_PUBLIC_ALCHEMY_ETHEREUM_HTTP }}
NEXT_PUBLIC_ALCHEMY_ETHEREUM_KEY: ${{ secrets.NEXT_PUBLIC_ALCHEMY_ETHEREUM_KEY }}
NEXT_PUBLIC_ALCHEMY_SEPOLIA_HTTP: ${{ secrets.NEXT_PUBLIC_ALCHEMY_SEPOLIA_HTTP }}
NEXT_PUBLIC_ALCHEMY_SEPOLIA_KEY: ${{ secrets.NEXT_PUBLIC_ALCHEMY_SEPOLIA_KEY }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
JWT_SECRET: ${{ secrets.JWT_SECRET }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
run: npx semantic-release
4 changes: 1 addition & 3 deletions lib/client/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -154,9 +154,7 @@ export const getCurrentNetworkHttpUrl = (chainId: number) => {
const httpUrl = getRpcHttpUrlForNetwork.get(chainId);

if (!httpUrl) {
if (process.env.NODE_ENV === "production")
throw new Error(`No RPC URL was defined for chain ID: ${chainId}`);
else console.error(`No RPC URL was defined for chain ID: ${chainId}`);
throw new Error(`No RPC URL was defined for chain ID: ${chainId}`);
}

return httpUrl;
Expand Down

0 comments on commit 0d996e9

Please sign in to comment.