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

Add pnpm caching support #278

Merged
merged 12 commits into from
Jul 20, 2021
Merged

Add pnpm caching support #278

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
162ddf7
Add pnpm caching support
jacobwgillespie Jun 30, 2021
f24e78e
Update __tests__/cache-save.test.ts
jacobwgillespie Jul 14, 2021
fdbc93e
Add disclaimer to pnpm action example, pin action
jacobwgillespie Jul 14, 2021
d278e78
Add logic to check that cache folder exists
jacobwgillespie Jul 14, 2021
399982b
Move existence check to cache-save
jacobwgillespie Jul 14, 2021
0453e51
Regenerate compiled files
jacobwgillespie Jul 14, 2021
e93556c
Mock fs.existsSync in tests
jacobwgillespie Jul 14, 2021
f452812
Unmock fs.existsSync after tests
jacobwgillespie Jul 15, 2021
3af302a
Switch to pnpm store path command
jacobwgillespie Jul 15, 2021
b96348a
Remove unused imports
jacobwgillespie Jul 15, 2021
4bc87b8
Bump e2e tests to 6.10.0
maxim-lobanov Jul 15, 2021
0ae03de
Reorder to npm, yarn, pnpm
jacobwgillespie Jul 15, 2021
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
Update __tests__/cache-save.test.ts 
Co-authored-by: Konrad Pabjan <konradpabjan@github.com>
  • Loading branch information
@jacobwgillespie @konradpabjan
jacobwgillespie and konradpabjan authored Jul 14, 2021
commit f24e78e0f6461d5f307b67ef629bcb2d60daf667
2 changes: 1 addition & 1 deletion __tests__/cache-save.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ describe('run', () => {
expect(setFailedSpy).not.toHaveBeenCalled();
});

it('should not save cache for npm', async () => {
it('should not save cache for pnpm', async () => {
inputs['cache'] = 'pnpm';
getStateSpy.mockImplementation(() => pnpmFileHash);
getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/pnpm`);
Expand Down