quarto-dev / quarto-dev/quarto-cli

Don't bundle JS anymore

Open
#4,955 1 comment 2 reactions 1 assignee View on GitHub

@cscheid is already working on this.

Since Mar 24, 2023.

deno early-in-release enhancement installers
Dominant language
JavaScript
Stars
6k
Forks
458
Avg merge
1d 9h
Merged PRs (30d)
41

Description

We no longer need to bundle JS now that deno has improved their caching behavior.

In addition, deno bundle is deprecated and doesn't work with npm specifiers, and is the reason we're stuck with 1.28.2.

On top of that, deno bundle is causing us continued grief with its bugs in cyclic dependencies.

So, let's get rid of it.

I think the main complication is figuring out how to build releases, specifically in macOS, where the notarizer is very particular about directory names. We might need to do a little bit of working around that.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.