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

Jump to bottom

fix npm re-download #217

Closed
wants to merge 2 commits into from
Closed

fix npm re-download #217

wants to merge 2 commits into from

Conversation

buwon
Copy link

@buwon buwon commented Jan 4, 2023

Added a copy of the npm folder. #214

@hayd
Copy link
Contributor

hayd commented Jan 6, 2023

Thanks! Is the gen/ directory no longer there?

@buwon
Copy link
Author

buwon commented Jan 6, 2023

Line 238 and line 240 are the same.

@pjroth
Copy link

pjroth commented Jan 7, 2023

I tested only the change in 076fe2a and it worked for me (no longer re-downloads on first lambda execution).

Thanks for this!

@buwon
Copy link
Author

buwon commented Jan 12, 2023

I get a permission error on deno.lock.

error: Read-only file system (os error 30)

It does not re-download, but it seems to overwrite deno.lock.
I have to copy deno.lock under /tmp.
I followed the README, but what is the problem?

@osak
Copy link

osak commented Jun 19, 2023

Any updates on this?

@luistio
Copy link

luistio commented Feb 19, 2024

any update on this one?

@buwon
Copy link
Author

buwon commented Feb 19, 2024

Sorry.
I'm not sure how deno works.
This method does not seem to be a clear solution.

@buwon buwon closed this Feb 19, 2024
@haruyuki-16278 haruyuki-16278 mentioned this pull request Mar 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@buwon @hayd @pjroth @osak @luistio