Skip to content

chore: format#229

Open
vegerot wants to merge 1 commit intoJerryZLiu:mainfrom
vegerot:pr229
Open

chore: format#229
vegerot wants to merge 1 commit intoJerryZLiu:mainfrom
vegerot:pr229

Conversation

@vegerot
Copy link
Contributor

@vegerot vegerot commented Mar 7, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 7, 2026 01:08
Copy link

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 formatting-only updates across UI tests, unit tests, and several core app/UI files to standardize indentation and line wrapping.

Changes:

  • Reindented XCTest UI/unit test files to consistent 2-space indentation.
  • Reformatted SwiftUI and core manager/service code (line wrapping, brace placement, argument formatting).
  • Minor whitespace/line-break adjustments in multi-line string literals and chained expressions.

Reviewed changes

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

Show a summary per file
File Description
Dayflow/DayflowUITests/DayflowUITestsLaunchTests.swift Indentation/whitespace formatting only.
Dayflow/DayflowUITests/DayflowUITests.swift Indentation/whitespace formatting only.
Dayflow/DayflowTests/TimeParsingTests.swift Added spacing + consistent indentation; no logic changes.
Dayflow/Dayflow/Views/UI/ChatView.swift Wrapped long SwiftUI calls and minor formatting in analytics props/key retrieval.
Dayflow/Dayflow/Core/Recording/StorageManager.swift Formatting around guard and chained filter/sort expressions.
Dayflow/Dayflow/Core/Analysis/AnalysisManager.swift Formatting for wrapped expressions and a wrapped function signature.
Dayflow/Dayflow/Core/AI/DashboardChatMemoryStore.swift Formatting of chained normalization expression.
Dayflow/Dayflow/Core/AI/ChatService.swift Formatting of function signature, multi-line string section, ternary wrapping, and wrapped if let call.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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