Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@aholstrup1
Copy link
Contributor

@aholstrup1 aholstrup1 commented Jul 29, 2025

Summary

This PR reduces the build time with 25-30 minutes by reducing the build graph depth from 3 to 2. In order to minimize the build graph we will:

  • Compile the System Application, Business Foundation and their test library apps in the same job.
  • Compile the test apps and run the tests in the subsequent job

This allows the "Build Apps (W1)" job to run at depth 2 instead of depth 3 which is what reduces the build time. Furthermore, it also moves up the compilation of the Business Foundation so if there are compilation issues in the Business Foundation the build will fail faster.

Before https://github.com/microsoft/BCApps/actions/runs/16591734810 (1h 11m)

After: https://github.com/microsoft/BCApps/actions/runs/16592812687 (46m)

Work Item(s)

Fixes AB#593466

@aholstrup1 aholstrup1 requested a review from a team as a code owner July 29, 2025 11:46
@github-actions github-actions bot added the Build: scripts & configs Build scripts and configuration files label Jul 29, 2025
@github-actions github-actions bot added this to the Version 27.0 milestone Jul 29, 2025
Copy link
Contributor

@haoranpb haoranpb left a comment

Choose a reason for hiding this comment

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

Should those folders be renamed?

@aholstrup1 aholstrup1 enabled auto-merge (squash) July 31, 2025 11:38
Copy link
Contributor

@Groenbech96 Groenbech96 left a comment

Choose a reason for hiding this comment

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

LGTM

@aholstrup1 aholstrup1 merged commit ba47e26 into main Aug 5, 2025
29 checks passed
@aholstrup1 aholstrup1 deleted the aholstrup/move_projects_around branch August 5, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build: scripts & configs Build scripts and configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants