Skip to content

fix(lint): to pass lint for some files#45

Merged
BinWang28 merged 1 commit intoMiroMindAI:miroflow-v0.3from
BinWang28:wb_dev_pass_lint
Sep 22, 2025
Merged

fix(lint): to pass lint for some files#45
BinWang28 merged 1 commit intoMiroMindAI:miroflow-v0.3from
BinWang28:wb_dev_pass_lint

Conversation

@BinWang28
Copy link
Member

fix(lint): to pass lint for some files

@BinWang28 BinWang28 requested review from Copilot and ntudy September 22, 2025 05:49
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

This PR applies linting fixes to multiple Python files to ensure they pass code style checks. The changes focus on formatting improvements including line length compliance, quote consistency, and trailing comma corrections.

Key changes:

  • Reformats long lines to comply with line length limits by breaking them across multiple lines
  • Standardizes string quotes from single to double quotes throughout the codebase
  • Removes unused variable assignments and adds missing trailing commas

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
utils/progress_check/check_futurex_progress.py Removes unused variable, reformats long lines for function calls and print statements
utils/prepare_benchmark/gen_xbench_ds.py Changes single quotes to double quotes and reformats long method chains
utils/prepare_benchmark/gen_futurex.py Adds missing trailing comma in dictionary
utils/extract_futurex_results.py Reformats long lines and function signatures, adds missing trailing comma

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@BinWang28 BinWang28 merged commit 55a27bd into MiroMindAI:miroflow-v0.3 Sep 22, 2025
1 check passed
Zhudongsheng75 pushed a commit to open-compass/MiroFlow that referenced this pull request Dec 27, 2025
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