Skip to content

Conversation

@chiranjeev13
Copy link
Member

@chiranjeev13 chiranjeev13 commented Dec 5, 2025

  • Updated the version of @inco/lightning to 'latest' in bun.lock and package.json.
  • Modified foundry.toml to specify test directory and patterns.
  • Enhanced SimpleConfidentialToken contract to require a fee for transfers.
  • Updated test cases for SimpleConfidentialToken to include fee handling and improved assertions.

Fixes https://github.com/Inco-fhevm/inco-monorepo/issues/1263

- Updated the version of @inco/lightning to 'latest' in bun.lock and package.json.
- Modified foundry.toml to specify test directory and patterns.
- Enhanced SimpleConfidentialToken contract to require a fee for transfers.
- Updated test cases for SimpleConfidentialToken to include fee handling and improved assertions.
"private": true,
"dependencies": {
"@inco/lightning": "0.6.8",
"@inco/lightning": "latest",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure If I want to have this pointing to latest instead of a specific release that we want to test

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Down the line, I’d prefer having this automated on every release so it always tests against the latest version. Maybe the better approach is to configure this in CI — not only to test against the latest release, but also to support a customizable environment parameter that lets us test against a specific release.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted to 0.7.0 for now

Copy link
Contributor

@kourtellosg kourtellosg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@chiranjeev13 chiranjeev13 merged commit 59c46bc into main Dec 8, 2025
1 check passed
@kourtellosg kourtellosg deleted the cm/update-foundry branch December 15, 2025 08:26
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.

3 participants