Skip to content

Include Node.js version in cache key #1171

Open
@clintonb

Description

@clintonb

Description:
The Node.js version should be included in the cache key. The major version number, (e.g., 22 in 22.12) should be used to avoid unnecessary invalidation when upgrading to patch or minor versions.

This is similar to #1008.

Justification:
Some dependencies need to be rebuilt for specific Node versions. Reusing the cache for an older version results in dependency installation failures.

Are you willing to submit a PR?
Yes.

Activity

linked a pull request that will close this issue on Dec 14, 2024
gowridurgad

gowridurgad commented on Dec 16, 2024

@gowridurgad
Contributor

Hello @clintonb ,
Thank you for this feature request. We will investigate it and get back to you as soon as we have some feedback.


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

Metadata

Metadata

Assignees

No one assigned

    Labels

    feature requestNew feature or request to improve the current logic

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Participants

    @clintonb@gowridurgad

    Issue actions

      Include Node.js version in cache key · Issue #1171 · actions/setup-node