Skip to content

Conversation

andrejrakic
Copy link
Collaborator

This PR adds CCT instructions to README file

@andrejrakic andrejrakic self-assigned this Jun 25, 2025
Copy link

@Copilot Copilot AI left a 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 adds instructions for using the Cross-Chain Token (CCT) standard to the README, guiding users through environment setup and test commands.

  • Added a new “CCT Standard” section to the Table of Contents and detailed instructions.
  • Introduced BASE_SEPOLIA_RPC_URL environment variable in prerequisites.
  • Provided CCT overview, example file references, and test commands.
Comments suppressed due to low confidence (1)

README.md:725

  • The filename in the square brackets (CCIPv1_5ForkBurnMintPoolFork.t.sol) does not match the linked file path (CCIPv1_5BurnMintPoolFork.t.sol). Please correct the reference so both match the actual test file name.
1. `getCCIPAdmin()` function in `MockERC20BurnAndMintToken` in [`CCIPv1_5ForkBurnMintPoolFork.t.sol`](./test/fork/CCIPv1_5BurnMintPoolFork.t.sol) file

Copy link
Collaborator

@zeuslawyer zeuslawyer left a comment

Choose a reason for hiding this comment

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

Thanks! left some comments and then good to merge!

To run CCT examples make sure you have the following environment variables set in your `.env` file:

```shell
ETHEREUM_SEPOLIA_RPC_URL=""
Copy link
Collaborator

Choose a reason for hiding this comment

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

is there a way to do this without adding another config env var for BASE? each additional var adds friction to the end user in terms of steps etc.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Swapped Base Sepolia for Avalanche Fuji

@andrejrakic andrejrakic merged commit a024e8e into main Jun 26, 2025
2 checks passed
@andrejrakic andrejrakic deleted the chore/add-cct-instructions branch June 26, 2025 21:23
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