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

Detect cached folders from multiple directories #735

Merged
merged 21 commits into from
Jun 21, 2023
Merged

Detect cached folders from multiple directories #735

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
02aa329
Add project-dir
dsame Apr 10, 2023
2d87f2f
Fix find lock file
dsame Apr 11, 2023
0b61567
Remove package-dir input
dsame Apr 13, 2023
9e025d3
format & resolve conflicts
dsame Apr 13, 2023
9b43de6
Add unit tests
dsame Apr 19, 2023
02b94f2
build dist
dsame Apr 24, 2023
8135560
Apply change request fixes
dsame Apr 24, 2023
a3d2aaf
handle non-dir cache-dependency-path
dsame May 4, 2023
d836439
bump cache version
dsame May 4, 2023
2b97a8f
run checks
dsame May 8, 2023
914a8e9
Handle globs in cacheDependencyPath
dsame May 11, 2023
5de08ea
refactor, introduce `cacheDependencyPathToProjectsDirectories`
dsame May 19, 2023
7406bf5
Changes requests
dsame May 22, 2023
9ddc512
Apply fixes
dsame May 30, 2023
d2b7e08
review fixes
dsame Jun 2, 2023
885f6b5
add e2e
dsame Jun 5, 2023
b8b9502
Add unique
dsame Jun 6, 2023
6aacde7
review updates
dsame Jun 8, 2023
3ef5272
review updates second stage
dsame Jun 14, 2023
17a4854
Review fixes 3
dsame Jun 16, 2023
e9e4944
imporve e2e tests
dsame Jun 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Review fixes 3
  • Loading branch information
@dsame
dsame committed Jun 16, 2023
commit 17a48541930c3b0d064ba15c28ea5eb95d63c00e
8 changes: 0 additions & 8 deletions __tests__/cache-save.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,14 +143,6 @@ describe('run', () => {
expect(getStateSpy).toHaveBeenCalledTimes(3);
expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
expect(debugSpy).toHaveBeenCalledTimes(0);
/*
expect(debugSpy).toHaveBeenCalledWith(
'Consumed yarn version is 2.2.3 (working dir: "")'
);
expect(debugSpy).toHaveBeenCalledWith(
'yarn\'s cache folder "/some/random/path/yarn2" configured for the root directory'
);
*/
expect(infoSpy).toHaveBeenCalledWith(
`Cache hit occurred on the primary key ${yarnFileHash}, not saving cache.`
);
Expand Down
1 change: 0 additions & 1 deletion dist/cache-save/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60516,7 +60516,6 @@ const getProjectDirectoriesFromCacheDependencyPath = (cacheDependencyPath) => __
const existingDirectories = cacheDependenciesPaths
.map(path_1.default.dirname)
.filter(util_1.unique())
.filter(fs_1.default.existsSync)
.filter(directory => fs_1.default.lstatSync(directory).isDirectory());
if (!existingDirectories.length)
core.warning(`No existing directories found containing cache-dependency-path="${cacheDependencyPath}"`);
Expand Down
1 change: 0 additions & 1 deletion dist/setup/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71299,7 +71299,6 @@ const getProjectDirectoriesFromCacheDependencyPath = (cacheDependencyPath) => __
const existingDirectories = cacheDependenciesPaths
.map(path_1.default.dirname)
.filter(util_1.unique())
.filter(fs_1.default.existsSync)
.filter(directory => fs_1.default.lstatSync(directory).isDirectory());
if (!existingDirectories.length)
core.warning(`No existing directories found containing cache-dependency-path="${cacheDependencyPath}"`);
Expand Down
1 change: 0 additions & 1 deletion src/cache-utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ const getProjectDirectoriesFromCacheDependencyPath = async (
const existingDirectories: string[] = cacheDependenciesPaths
.map(path.dirname)
.filter(unique())
.filter(fs.existsSync)
.filter(directory => fs.lstatSync(directory).isDirectory());

if (!existingDirectories.length)
Expand Down