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
Enable caching by default #321
Conversation
In order to avoid vague `default` value in many places
In order to hande empty 'cache' input
It is needed to enable cache
@@ -13,3 +13,5 @@ export const supportedPackageManagers: SupportedPackageManagers = { | |||
cacheFolderCommandList: ['go env GOMODCACHE', 'go env GOCACHE'] | |||
} | |||
}; | |||
|
|||
export const getCurrentPackageManager = (): string => 'default'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export const getCurrentPackageManager = (): string => 'default'; | |
export const getDefaultPackageManager = (): string => 'default'; |
@@ -14,7 +14,7 @@ inputs: | |||
default: ${{ github.server_url == 'https://github.com' && github.token || '' }} | |||
cache: | |||
description: Used to specify whether caching is needed. Set to true, if you'd like to enable caching. | |||
default: false | |||
required: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
required: false | |
default: true | |
required: false |
@@ -73,3 +78,16 @@ export function isCacheFeatureAvailable(): boolean { | |||
); | |||
return false; | |||
} | |||
|
|||
export async function isCacheEnabled() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might split it into two functions:
isCacheEnabled
and isCachePathsExists
The PR is aborted due to #332 |
Description:
Change default value for
cache
inputRelated issue:
Add link to the related issue.
Check list: