Skip to content

chore(task): add test for empty new line#203

Merged
fengmk2 merged 2 commits intomainfrom
09-25-chore_task_remove_unuse_empty_line
Sep 25, 2025
Merged

chore(task): add test for empty new line#203
fengmk2 merged 2 commits intomainfrom
09-25-chore_task_remove_unuse_empty_line

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Sep 25, 2025

No description provided.

@fengmk2 fengmk2 changed the title chore(task): remove unuse empty line fix(vite_task): remove extra newlines in task output Sep 25, 2025
Copy link
Copy Markdown
Member Author

fengmk2 commented Sep 25, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fengmk2 fengmk2 changed the title fix(vite_task): remove extra newlines in task output fix(task): remove extra newlines in task output Sep 25, 2025
@fengmk2 fengmk2 self-assigned this Sep 25, 2025
@fengmk2 fengmk2 marked this pull request as ready for review September 25, 2025 06:58
@fengmk2 fengmk2 changed the base branch from 09-25-feat_task_support_fast_failure_exit to graphite-base/203 September 25, 2025 07:05
@fengmk2 fengmk2 force-pushed the 09-25-chore_task_remove_unuse_empty_line branch from 844aafe to e81cb21 Compare September 25, 2025 07:05
@fengmk2 fengmk2 changed the base branch from graphite-base/203 to main September 25, 2025 07:05
Copilot AI review requested due to automatic review settings September 25, 2025 07:17
Copy link
Copy Markdown
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 removes unnecessary blank lines from task execution output to improve readability. The changes address extra newlines that were appearing after task executions and in various output scenarios.

  • Remove redundant println!() statement in task execution code
  • Update snapshot tests to reflect cleaner output formatting without extra blank lines
  • Ensure consistent output formatting across different task execution scenarios

Reviewed Changes

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

File Description
crates/vite_task/src/schedule.rs Remove extra println statement that was adding unnecessary newlines
packages/cli/snap-tests/*/snap.txt Update test snapshots to reflect removal of extra blank lines in task output

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

@fengmk2 fengmk2 force-pushed the 09-25-chore_task_remove_unuse_empty_line branch from cde5845 to 0bf3cde Compare September 25, 2025 07:39
@fengmk2 fengmk2 changed the title fix(task): remove extra newlines in task output chore(task): add test for empty new line Sep 25, 2025
@fengmk2 fengmk2 merged commit b8e0dc3 into main Sep 25, 2025
9 checks passed
Copy link
Copy Markdown
Member Author

fengmk2 commented Sep 25, 2025

Merge activity

@fengmk2 fengmk2 deleted the 09-25-chore_task_remove_unuse_empty_line branch September 25, 2025 07:54
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.

3 participants