Skip to content

chore: store pre-built JS in source tree#165

Merged
Vtec234 merged 5 commits into
mainfrom
git-js
Mar 27, 2026
Merged

chore: store pre-built JS in source tree#165
Vtec234 merged 5 commits into
mainfrom
git-js

Conversation

@Vtec234

@Vtec234 Vtec234 commented Mar 27, 2026

Copy link
Copy Markdown
Collaborator

This PR stores pre-built JavaScript code directly in the source tree instead of asking consumers to fetch it via a combination of Lake cloud releases and (in mathlib) mathlib4/Cache.

Fetching and reuse of this code has unfortunately proved to be unreliable, causing much anguish and gnashing of teeth. This change is not foolproof, in particular caching/reuse issues can still cause the build to fail, but should hopefully reduce the number of components involved.

@Vtec234
Vtec234 merged commit 1d1093b into main Mar 27, 2026
4 checks passed
@Vtec234
Vtec234 deleted the git-js branch March 27, 2026 20:24
@Vtec234
Vtec234 restored the git-js branch March 27, 2026 21:25
@Vtec234

Vtec234 commented Mar 27, 2026

Copy link
Copy Markdown
Collaborator Author

(Adapting mathlib may take a while, reverted this on main for now so that toolchain updates can succeed in the meantime.)

mathlib-bors Bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 27, 2026
Vtec234 added a commit that referenced this pull request Mar 30, 2026
* chore: move JS build dir to widget/js/

* doc: new pre-built strategy

* chore: update widget/js

* ci: adjust to changes

* chore: bump @leanprover-community/proofwidgets4
mathlib-bors Bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 30, 2026
This PR removes special support for ProofWidgets4 cloud releases. See leanprover-community/ProofWidgets4#165.
Rob23oba pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Mar 31, 2026
aditya-ramabadran pushed a commit to aditya-ramabadran/mathlib4 that referenced this pull request Apr 1, 2026
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.

1 participant