Skip to content

Conversation

cscheid
Copy link
Collaborator

@cscheid cscheid commented Oct 1, 2024

This PR updates Deno to 1.46.3 along with our dependencies, stdlib, build scripts, etc.

@cscheid
Copy link
Collaborator Author

cscheid commented Oct 2, 2024

@cderv In case you're curious, the test failure we're seeing is because of this:

https://github.com/quarto-dev/quarto-cli/actions/runs/11147504574/job/30982013277?pr=10940#step:26:226

Note how we're passing the import map from the github cache. The github cache uses the latest release, and now these import maps are no longer compatible with each other.

@cscheid cscheid force-pushed the chore/deno-update-2024-09 branch from bf4390d to 7408297 Compare October 2, 2024 17:16
@cscheid
Copy link
Collaborator Author

cscheid commented Oct 2, 2024

I'm going to merge this with the failing test (because the test needs a release to work, and it's easier to fix in this way.)

@cscheid cscheid merged commit 23ea14b into main Oct 2, 2024
46 of 47 checks passed
@cscheid cscheid deleted the chore/deno-update-2024-09 branch October 2, 2024 18:28
@cderv
Copy link
Collaborator

cderv commented Oct 2, 2024

because the test needs a release to work

I am not just to follow this really - our tests run with dev version AFIU. But we can discuss this tomorrow

@cscheid
Copy link
Collaborator Author

cscheid commented Oct 2, 2024

Look at where the import map is coming from here: https://github.com/quarto-dev/quarto-cli/actions/runs/11149485730/job/30988488885#step:26:226

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants