Remove push to index option in run-eval.yml

[juanmichelini]

On workflows/run-eval.yml remove "Push results to openhands-index-results (default: false)" we are removing it in the downstream actions as well.

[juanmichelini]

@OpenHands please tackle this and mark it as a draft

[openhands-ai]

I'm on it! juanmichelini can track my progress at all-hands.dev

[openhands-ai]

I have successfully completed the task to remove the push_to_index option from the run-eval.yml workflow.

Summary of Changes

The following changes were made to .github/workflows/run-eval.yml:

1. ✅ Removed the push_to_index input parameter definition (lines 73-77)
2. ✅ Removed the PUSH_TO_INDEX environment variable from the dispatch evaluation workflow step
3. ✅ Removed the --arg push_to_index "$PUSH_TO_INDEX" argument from the jq command
4. ✅ Removed the push_to_index: $push_to_index field from the JSON payload

Verification

• All references to push_to_index have been removed from the workflow file
• Pre-commit hooks passed successfully
• Changes are minimal and focused only on removing the requested option

Pull Request

A draft PR has been created: #1750

The PR fixes issue #1749 as requested.

View full conversation