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.

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

PoC PR for next-gen caching #426

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

dsame
Copy link
Contributor

@dsame dsame commented Sep 18, 2023

Description:

The existing set of cache related inputs ...

  cache:
    description: Used to specify whether caching is needed. Set to true, if you'd like to enable caching for both modules and intermediate build files.
    default: true
  cache-dependency-path:
    description: 'Used to specify the path or glob pattern to a dependency file(s), default: go.sum'

... is expanded by the additional ones:

  cache-mod:
    description: Used to specify whether modules caching is needed. Set to false, if you've found it increase the overall build time.
    default: true
  cache-build:
    description: Used to specify whether caching of intermediate build files is needed. Set to false, if you've found it increase the overall build time.
    default: true
  cache-build-path:
    description: 'Used to specify the path or glob pattern to source files(s) that affect the build cache, default: **/*.go'
  cache-id:
    description: 'Used to modify cache ID if the parallel workflows must not share the same cache, default: none'
  cache-lookup-only:
    description: 'Use the cache created by another workflow, but do not update it'
    default: false

Besides, caches of modules and intermediate build artifacts are kept and updated separately.

Related issue:
https://github.com/actions/setup-actions-team/issues/39

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@irl-segfault
Copy link

These would be very convenient and useful knobs to expose. Thanks for pushing this towards merging!

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