Skip to content

Conversation

vargaat
Copy link
Collaborator

@vargaat vargaat commented Oct 6, 2025

What's new?

  • Refactored large flatMap chains that caused very slow type checking. One of these checks took 8 secs on my M1. No behavior changes.
  • I made 3 before-after measurements and on average the compile was 8 seconds faster (from 146.3 to 138.4) after the refactoring.

[ignore-commit-lint]

@vargaat vargaat self-assigned this Oct 6, 2025
@inst-danger
Copy link
Contributor

Builds

No apps were built for this pull request.

To trigger app builds, include a line starting with builds: followed by app names (Student, Teacher, Parent, or All) in your pull request's message.

Commit: Another refactor. (3f8dfe5)
Built At: Oct 06 14:56 CEST (10/06 06:56 AM MDT)

@vargaat vargaat requested a review from szabinst October 6, 2025 13:07
@inst-danger
Copy link
Contributor

Warnings
⚠️ One or more files are below the minimum test coverage 50%
⚠️ The total test coverage is below the minimum 90%
Coverage New % Master % Delta
Canvas iOS 79.09% 79.09% -0%
Core/Core/Common/Extensions/Combine/SubjectExtensions.swift 0% 0% 0%
Core/Core/Features/Planner/Model/API/APICalendarEventSubAssignment.swift 0% 0% 0%
Core/Core/Features/Enrollments/GetEnrollments.swift 48.81% 48.81% 0%
Horizon/Horizon/Sources/Features/LearningObjects/Assignment/AssignmentDetails/View/LTIQuiz/OnAppear.swift 0% 0% 0%
Horizon/Horizon/Sources/Common/View/EmbeddedWebPage/HEmbeddedWebPageContainerViewModel.swift 0% 0% 0%
Horizon/Horizon/Sources/Features/LearningObjects/Assignment/AssignmentDetails/View/LTIQuiz/DidSubmitQuiz.swift 0% 0% 0%
Horizon/Horizon/Sources/Features/Inbox/Attachment/AttachmentViewModel.swift 0% 0% 0%
Horizon/Horizon/Sources/Features/Notebook/Common/Data/UpdateRedwoodNote.swift 0% 0% 0%
Horizon/Horizon/Sources/Features/LearningObjects/Assignment/AssignmentDetails/Data/SubmissionProperties.swift 0% 0% 0%
Horizon/Horizon/Sources/Common/Data/LearningObjectType.swift 0% 0% 0%

Generated by 🚫 dangerJS against 3f8dfe5

Copy link
Contributor

@petkybenedek petkybenedek left a comment

Choose a reason for hiding this comment

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

QA +1

@vargaat vargaat merged commit f17e095 into master Oct 13, 2025
3 checks passed
@vargaat vargaat deleted the hackday/optimize-compile-time branch October 13, 2025 11:48
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.

5 participants