Skip to content

feat: add env variables properly and proper documentations#909

Merged
ogazboiz merged 4 commits into
LabsCrypt:mainfrom
dinahmaccodes:issuenew
Jun 29, 2026
Merged

feat: add env variables properly and proper documentations#909
ogazboiz merged 4 commits into
LabsCrypt:mainfrom
dinahmaccodes:issuenew

Conversation

@dinahmaccodes

Copy link
Copy Markdown
Contributor

PR Description

This PR fixes documentation inconsistencies, deployment script issues, adds CI size budget enforcement, and configures code ownership for review routing.

Changes:

  • Fixed environment variable names in ARCHITECTURE.md to match backend code (STREAM_CONTRACT_ID, lowercase STELLAR_NETWORK)
  • Fixed deploy.sh to prevent clobbering exported environment variables with empty defaults
  • Added WASM size budget check (200KB) to CI workflow to prevent size regressions
  • Added CODEOWNERS file for automatic PR review assignment

Checklist:

Closes:
Closes #900
Closes #887
Closes #896
Closes #904

@ogazboiz ogazboiz merged commit 60a0b9f into LabsCrypt:main Jun 29, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment