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.

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

DENO_DIR cache *maybe* no longer works #116

Open
hayd opened this issue Nov 21, 2020 · 3 comments
Open

DENO_DIR cache *maybe* no longer works #116

hayd opened this issue Nov 21, 2020 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@hayd
Copy link
Contributor

hayd commented Nov 21, 2020

Previously

import type {
  Context,
  APIGatewayProxyEvent
} from "https://deno.land/x/lambda/mod.ts";

etc.

the deps and gen were correctly cached (from the layer and the function code), this seems to no longer be the case.

It's unclear to me if it's a zipping issue or due the filenames being renamed (now there's hashes in the name).


I'm not 100% sure this is an issue, but have a suspicion... needs investigate (and ideally a test).

@hayd hayd added the help wanted Extra attention is needed label Nov 21, 2020
@hayd hayd changed the title DENO_DIR cache no longer works DENO_DIR cache *maybe* no longer works Nov 21, 2020
@hayd
Copy link
Contributor Author

hayd commented Dec 18, 2020

Specifically deno.land/x/lambda should not be fetched during tests?

@hayd
Copy link
Contributor Author

hayd commented Jun 4, 2021

So, previously the https://deno.land/x/lambda/mod.ts and types.d.ts were prepopulated from the current dir (avoiding the fetch). This is no longer the case :/

I am not sure the best solution here... perhaps simply not using https://deno.land/x/lambda/mod.ts ? 😢

@hayd
Copy link
Contributor Author

hayd commented Nov 3, 2021

Use deno-lambda with docker is my recommendation for this.

Deno uses more opaque/clever caching for local file compiled assets than it once did. I haven't looked into this in a while / am not sure how much/if any is exposed so mv assets could work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant