Skip to content

Fixed absolute path cache miss in workflow-testing. #1115

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
Open
techexplorer1231 wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Fixed absolute path cache miss in workflow-testing. #1115

techexplorer1231 wants to merge 5 commits into from
+19 −6

Conversation

@techexplorer1231
Copy link

@techexplorer1231 techexplorer1231 commented Oct 10, 2023

Fixed absolute path cache miss in workflow-testing.

  • Provided friendPaths using built-in property that is marked @internal.
  • When you need to reference an artifact from another project, the best practice is to use Variant-aware sharing of artifacts between projects. With this strategy we can avoid directly referencing the configuration of another project and Gradle will set up proper task dependencies.

@CLAassistant
Copy link

CLAassistant commented Oct 10, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

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

Reviewers

@zach-klippenstein zach-klippenstein Awaiting requested review from zach-klippenstein zach-klippenstein is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@RBusarow RBusarow

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@techexplorer1231 @CLAassistant @RBusarow