Skip to content
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

[CCIP-5323] Remove unused operatorforwarder contracts and their associated code. #16681

Draft
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

pavel-raykov
Copy link
Collaborator

@pavel-raykov pavel-raykov changed the title Remove unused operator contracts and their associated code. Remove unused operatorforwarder contracts and their associated code. Mar 6, 2025
Copy link
Contributor

github-actions bot commented Mar 6, 2025

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@pavel-raykov pavel-raykov changed the title Remove unused operatorforwarder contracts and their associated code. [CCIP-5323] Remove unused operatorforwarder contracts and their associated code. Mar 6, 2025
Copy link
Contributor

github-actions bot commented Mar 6, 2025

Solidity Review Jira issue

Hey! We have taken the liberty to link this PR to a Jira issue for Solidity Review.

This is a new feature, that's currently in the pilot phase, so please make sure that the linkage is correct. In a contrary case, please update it manually in JIRA and replace Solidity Review issue key in the changeset file with the correct one.
Please reach out to the Test Tooling team and notify them about any issues you encounter.

Any changes to the Solidity Review Jira issue should be reflected in the changeset file. If you need to update the issue key, please do so manually in the following changeset file: contracts/.changeset/shiny-toys-think.md

This PR has been linked to Solidity Review Jira issue: CCIP-3966

@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@pavel-raykov pavel-raykov marked this pull request as ready for review March 6, 2025 16:06
@pavel-raykov pavel-raykov requested review from a team as code owners March 6, 2025 16:06
@pavel-raykov pavel-raykov requested review from cedric-cordenier, jmank88 and RensR and removed request for cedric-cordenier March 6, 2025 16:06
@jmank88
Copy link
Contributor

jmank88 commented Mar 6, 2025

I am surprised that this one is unused. Are we sure that it is dead? It may have just been delayed.

@pavel-raykov
Copy link
Collaborator Author

I am surprised that this one is unused. Are we sure that it is dead? It may have just been delayed.

the files were added ~ 1 year ago and are not referenced from any other repository in our org. I would defer to the owners if it is fine to remove them.

@pavel-raykov pavel-raykov marked this pull request as draft March 6, 2025 19:20
@pavel-raykov pavel-raykov removed the request for review from jmank88 March 6, 2025 19:20
@pavel-raykov pavel-raykov requested review from jmank88, RensR and cawthorne and removed request for RensR, jmank88 and cawthorne March 6, 2025 19:20
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