refactor: load TON chain configuration from yaml store #2185
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
Co-Authored-By: Jayant Krishnamurthy <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
Add "(aside)" to your comment to have me ignore it. |
Co-Authored-By: Jayant Krishnamurthy <[email protected]>
README.md
Outdated
| @@ -108,6 +108,7 @@ after `--`, for instance you could run `pnpm test -- --concurrency 2`. | |||
| checks, as well as whatever other code checks any packages support. | |||
| - `pnpm fix`: Run auto fixes, including reformatting code and auto-fixing lint | |||
| rules where possible. | |||
| - `pnpm turbo build`: Build all packages in the monorepo using turborepo. | |||
There was a problem hiding this comment.
let's remove this line, there shouldn't usually be a good reason to build all packages in the monorepo with turbo -- instead we should be doing things like linting / testing / starting / etc and let turbo build what needs to be built.
There was a problem hiding this comment.
cc @jayantk I'm not sure how we should be updating Devin's PRs....
There was a problem hiding this comment.
it might understand github comments, not sure. we'll find out though.
I think this might be a setup problem in the repo for contract_manager, because pnpm build:ci doesn't seem to work there.
There was a problem hiding this comment.
meta comment is i do think a bit more docs here would be useful. based on these directions, I don't know how to run scripts or anything. Last time i worked with contract manager i spent some time digging around to figure this out.
Co-Authored-By: Jayant Krishnamurthy <[email protected]>
contract_manager/package.json
Outdated
| "build": "tsc", | ||
| "build": "pnpm run build:cjs && pnpm run build:esm", | ||
| "build:cjs": "tsc -p tsconfig.json", | ||
| "build:esm": "tsc -p tsconfig.esm.json", |
There was a problem hiding this comment.
This should probably get reverted, this needs more testing before we should ship something like it
contract_manager/tsconfig.esm.json
Outdated
| "module": "esnext", | ||
| "outDir": "./lib/esm" | ||
| } | ||
| } |
devin-ai-integration
bot
force-pushed
the
devin/1733949624-ton-chain-config-from-yaml
branch
from
e53a3fb to
58f900b
Compare
This PR updates the TON contract upgrade script to load chain configuration from yaml files instead of hardcoding RPC URLs.
Changes:
Testing:
Link to Devin run: https://app.devin.ai/sessions/a73a806f601a44538f8e9260208f22cf