Skip to content

Conversation

@MarcusWentz
Copy link
Contributor

Description

Autopilot flag NATIVE_PRICE_ESTIMATORS set with 2 different values in docker-compose.fork.yml.

Changes

  • Remove duplicate NATIVE_PRICE_ESTIMATORS from docker-compose

Related Issues

#4063

@MarcusWentz MarcusWentz requested a review from a team as a code owner January 18, 2026 19:57
@github-actions
Copy link

github-actions bot commented Jan 18, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly removes a duplicate NATIVE_PRICE_ESTIMATORS environment variable in docker-compose.fork.yml, which improves configuration clarity. However, I've noticed that the remaining NATIVE_PRICE_ESTIMATORS variable appears to be incorrectly formatted and would likely cause a parsing error at runtime. I've left a comment with a suggestion to fix this related issue. Addressing this would make the configuration fully correct.

@squadgazzz
Copy link
Contributor

@MarcusWentz, please accept the CLA.

@MarcusWentz
Copy link
Contributor Author

@squadgazzz

I have read the CLA Document and I hereby sign the CLA

@squadgazzz
Copy link
Contributor

@MarcusWentz since the CI still fails, I assume the message should only contain the following text, without mentioning anyone:

I have read the CLA Document and I hereby sign the CLA

@MarcusWentz
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 19, 2026
@jmg-duarte jmg-duarte enabled auto-merge January 19, 2026 17:42
@jmg-duarte jmg-duarte added this pull request to the merge queue Jan 19, 2026
Merged via the queue into cowprotocol:main with commit dbd7f7f Jan 19, 2026
19 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants