Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix build cache issues and improve task configuration #3368

Closed
wants to merge 1 commit into from
Closed

Fix build cache issues and improve task configuration #3368

wants to merge 1 commit into from

Conversation

mondrejko
Copy link
Contributor

  • Identified and corrected tasks missing input and output values, improving cache efficiency.
  • Added missing path sensitivity configuration to prevent cache misses from differences in absolute file paths.
  • Added explicit log messages for tasks that are uncacheable.
  • Resolved overlapping task outputs issue: generatePackageInfo task now outputs to a build directory, with a finalizer task copying results to the source directory.

@mondrejko
Fix build cache issues and improve task configuration
8b1622f 
- Identified and corrected tasks missing input and output values, improving cache efficiency.
- Added missing path sensitivity configuration to prevent cache misses from differences in absolute file paths.
- Added explicit log messages for tasks that are uncacheable.
- Resolved overlapping task outputs issue: generatePackageInfo task now outputs to a build directory, with a finalizer task copying results to the source directory.
@lukasz-jarocki-sonarsource
Copy link
Member

Many thanks for the contribution, we are looking into it.

@lukasz-jarocki-sonarsource
Copy link
Member

lukasz-jarocki-sonarsource commented Feb 18, 2025 •
edited
Loading

Apologies for the delay but your commit has been approved and merged to the upstream repository. It will be visible in the public repository in 24 hours.

Parts of the commit (namely changes in the main build.gradle file) had to be removed as they were not compatible with our private code.

I will close this pull request now, according to our process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lukasz-jarocki-sonarsource lukasz-jarocki-sonarsource lukasz-jarocki-sonarsource approved these changes

Assignees

@lukasz-jarocki-sonarsource lukasz-jarocki-sonarsource

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mondrejko @lukasz-jarocki-sonarsource