-
Notifications
You must be signed in to change notification settings - Fork 15
(chore): Move CRE subfolders into versioned folders #327
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
Conversation
|
👋 justinkaseman, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR reorganizes the CRE (Chainlink Runtime Environment) contract structure by consolidating scattered subfolders into versioned directories (v1, v2, and dev), simplifying the project layout and build configuration.
Changes:
- Consolidated contracts and tests from
keystone/,workflow/, andshardconfig/folders intov1/,v2/, anddev/versioned directories - Updated import paths across test files to reflect the new directory structure
- Simplified build scripts and foundry profiles to align with the new organization
Reviewed changes
Copilot reviewed 50 out of 254 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| contracts/cre/src/v2/README.md | Added README describing CRE Mainline Contracts for v2 |
| contracts/cre/src/v1/README.md | Added README describing Keystone/ST3 Contracts for v1 |
| contracts/cre/src/v1/test/KeystoneForwarder/*.sol | Updated import paths to reflect moved source files (e.g., ../KeystoneForwarder.sol → ../../KeystoneForwarder.sol) |
| contracts/cre/src/v1/test/CapabilitiesRegistry/*.sol | Updated import paths for moved CapabilitiesRegistry and interface files |
| contracts/cre/src/dev/test/*.sol | Updated import paths for dev contracts to new structure |
| contracts/cre/src/dev/*.sol | Updated imports for IReceiver and IRouter interfaces to point to v1 location |
| contracts/cre/scripts/compile_all_* | Removed old compilation scripts for keystone/workflow/shardconfig; added new v1/v2/dev scripts |
| contracts/cre/gobindings/*/go_generate.go | Removed old gobindings generation files; reorganized to v1/v2/dev structure |
| contracts/cre/gas-snapshots/*.gas-snapshot | Removed old snapshots; added new v1/v2/dev snapshots with updated test names |
| contracts/cre/foundry.toml | Updated foundry profiles from workflow-v1/v2, keystone, shardconfig to v1/v2/dev |
| contracts/cre/GNUmakefile | Updated ALL_FOUNDRY_PRODUCTS from "keystone workflow" to "dev v1 v2" |
| .github/workflows/solidity-foundry.yml | Updated CI matrix to use dev/v1/v2 instead of workflow/keystone |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
✅ API Diff Results - No breaking changes |
a33ec04 to
2532f70
Compare
2532f70 to
4a72cdf
Compare
No description provided.