Repository navigation
Update to PureScript 0.15.17 and package set 82.1.0 - #324
Merged
Merged
Conversation
- Bump purescript to 0.15.17 and the Stackage resolver from lts-20.26 to lts-23.18 (GHC 9.2.8 -> 9.8.4), matching purescript-0.15.17's own stack.yaml. The extra-deps now mirror upstream; the old process, Cabal, protolude, and happy pins and the these flag are satisfied by the new snapshot and GHC. - Fix server/Main.hs for -Werror under the newer snapshot: drop a now redundant liftIO import and use scotty's queryParam instead of the deprecated param for /search. - Update to registry package set 82.1.0 using the steps in RELEASE.md. Exclude the new packages that cannot run in the playground (mycroft, mycroft-z3-wasm, node-buffer-blob, node-sqlite, yoga-scylladb, and their dependents baskerville-mycroft, env, yoga-sqlite-om) and drop elmish-time-machine, which left the set. - Update SharedConfig.purs and the client's purescript dev dependency. Memory at production RTS settings (-M3G): cold boot compiles all 3,901 modules with 1,200 MB peak residency; warm restart peaks at 1,896 MB.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates Try PureScript to PureScript 0.15.17 and registry package set 82.1.0, which targets compiler 0.15.17.
Changes
Bump
purescriptto 0.15.17 and the Stackage resolver fromlts-20.26tolts-23.18(GHC 9.2.8 → 9.8.4), matching purescript-0.15.17's ownstack.yaml. The extra-deps now match upstream's; the oldprocess,Cabal,protolude, andhappypins and thetheseflag are covered by the new snapshot and GHC.server/Main.hs: two-Werrorfixes for the newer snapshot. Drop an import that is now redundant (liftIO), and use scotty 0.22'squeryParaminstead of the deprecatedparamfor/search, which is aGET ?q=request.Update to package set 82.1.0 using the RELEASE.md steps: 501 packages installed, up from 492. Eight packages new to the set can't run in the playground and are added to
staging/excluded-packages.txt:mycroft,mycroft-z3-wasm,node-buffer-blob,node-sqlite,yoga-scylladbbaskerville-mycroft,env,yoga-sqlite-omNo package already in the set is newly affected.
elmish-time-machineleft the set, so its line is removed.Regenerate
SharedConfig.purs, bump the client'spurescriptdev dependency to^0.15.17, and add a changelog entry.Memory check (production RTS flags,
-M3G)Verification
stack buildsucceeds on GHC 9.8.4 with-Wall -Werror.Client:
npm run build,test,build:production,bundle:production, and the SharedConfig version check pass.All
client/examplescompile with 0 warnings against the local server and run in the UI with the expected output.Attached derive clauses compile and run in the UI. This module fails to parse on production today and works with this branch:
Attached
derive newtypeis rejected with a parse error, as the release notes describe.spago fetch --pure(the deploy start script) succeeds from a fresh cache with the new lock file.