Skip to content

BT-939 clean erc 3643 repo structure #249

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

pgonday
Copy link

@pgonday pgonday commented Jun 24, 2025

No description provided.

Joachim-Lebrun and others added 6 commits June 10, 2025 17:00
* 🔧 update OZ dependencies

* 🚨 lint fix
## 📋 Summary
Modernizing development infrastructure and improving test coverage for the T-REX protocol.

## 🔧 Updates
- **Hardhat**: `2.22.12` → `2.24.2` (latest bug fixes and performance improvements)
- **Solidity**: `0.8.27` → `0.8.30` (critical ABI-encoding fixes for calldata tuples/structs)
- **Node.js**: `18` → `20` in GitHub workflows (extended LTS support, better performance)
- **NPM**: `@tokenysolutions` → `@erc3643org` (changed the npm org on which new versions will be published)

## 🧪 Test Coverage
- ✅ **Modular Compliance**: Achieved full test coverage with comprehensive edge cases
- 🔄 **In Progress**: Working towards 100% test coverage across all remaining contracts

## ✅ Validation
- All existing tests pass with updated versions
- No breaking changes or regressions detected
- ABI-encoding operations validated with struct parameters
- CI/CD pipeline updated and functioning

## 🎯 Next Steps
- Complete full test coverage for all contracts
- Performance benchmarking with new versions

## ⚠️ Caution
This PR is built on top of #61 Therefore make sure that the previous PR changes are accepted before merging the current PR
@pgonday pgonday marked this pull request as draft June 24, 2025 08:18
@pgonday pgonday closed this Jun 24, 2025
@Joachim-Lebrun Joachim-Lebrun deleted the BT-939-clean-erc-3643-repo-structure branch June 24, 2025 09:40
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.

2 participants