Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: load store once per script with configurable path #2201

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Refactor Store Initialization

This PR refactors the contract manager to load the store once at the beginning of each script, with a configurable store directory path that defaults to the location of DefaultStore.

Changes

  • Added createStore() utility function in store.ts
  • Updated all scripts to use createStore() instead of DefaultStore
  • Added --store-dir CLI option to all scripts
  • Updated common.ts utility functions to accept Store parameter

Testing

  • Ran build and linting checks
  • Verified scripts compile successfully
  • Tested with both default and custom store paths

Link to Devin run: https://app.devin.ai/sessions/c2abc85d940e490b8c412681444a83ed

Copy link

vercel bot commented Dec 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2024 3:50pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2024 3:50pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2024 3:50pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
component-library ⬜️ Ignored (Inspect) Visit Preview Dec 17, 2024 3:50pm
insights ⬜️ Ignored (Inspect) Visit Preview Dec 17, 2024 3:50pm

Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants