Skip to content

Comments

enabling test#1724

Open
yashnevatia wants to merge 1 commit intomainfrom
yn-ci-test
Open

enabling test#1724
yashnevatia wants to merge 1 commit intomainfrom
yn-ci-test

Conversation

@yashnevatia
Copy link
Collaborator

@yashnevatia yashnevatia commented Feb 21, 2026 •
edited
Loading

core ref: f862f07301aa66772b8f85eb6d7c8e15bdf9eedd

@yashnevatia yashnevatia requested a review from a team as a code owner February 21, 2026 13:39
Copilot AI review requested due to automatic review settings February 21, 2026 13:39
@yashnevatia yashnevatia requested a review from a team as a code owner February 21, 2026 13:39
@github-actions
Copy link

github-actions bot commented Feb 21, 2026

👋 yashnevatia, 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!

Copilot AI reviewed Feb 21, 2026
View reviewed changes
Copy link
Contributor

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

Re-enables CCIP Solana messaging integration tests in the GitHub Actions integration-test matrix so they run again as part of the standard CI workflow.

Changes:

  • Uncomments/enables Test_CCIPMessaging_EVM2Solana in the workflow matrix.
  • Uncomments/enables Test_CCIPMessaging_Solana2EVM in the workflow matrix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.github/workflows/ccip-integration-test.yml
@@ -63,14 +63,14 @@ jobs:
run: "^Test_CCIPMessaging_EVM2EVM$"
timeout: 20m
# Tests broken as of 2026-02-20
Copy link

Copilot AI Feb 21, 2026

Choose a reason for hiding this comment

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

The comment says these tests are broken as of 2026-02-20, but the previously-disabled Solana messaging tests are now enabled below. Please remove or update this comment to reflect the current status (e.g., specify which tests are still broken/flaky, or update the date/reason).

Suggested change
# Tests broken as of 2026-02-20
# Solana messaging tests

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@winder winder Awaiting requested review from winder winder is a code owner automatically assigned from smartcontractkit/ccip-offchain

@matYang matYang Awaiting requested review from matYang matYang is a code owner automatically assigned from smartcontractkit/ccip-offchain

@makramkd makramkd Awaiting requested review from makramkd makramkd is a code owner automatically assigned from smartcontractkit/ccip-offchain

@agusaldasoro agusaldasoro Awaiting requested review from agusaldasoro agusaldasoro is a code owner automatically assigned from smartcontractkit/ccip-offchain

@ogtownsend ogtownsend Awaiting requested review from ogtownsend ogtownsend is a code owner automatically assigned from smartcontractkit/ccip-offchain

@tt-cll tt-cll Awaiting requested review from tt-cll tt-cll is a code owner automatically assigned from smartcontractkit/ccip-offchain

@chris-de-leon-cll chris-de-leon-cll Awaiting requested review from chris-de-leon-cll chris-de-leon-cll is a code owner automatically assigned from smartcontractkit/ccip-offchain

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yashnevatia