Skip to content

fix: peer-dependencies and target for JS codegen - #23211

Open
phillip-cashen wants to merge 1 commit into
digital-asset:mainfrom
phillip-cashen:phol-cashen/fix-peer-dependecy-and-target-codegen
Open

fix: peer-dependencies and target for JS codegen#23211
phillip-cashen wants to merge 1 commit into
digital-asset:mainfrom
phillip-cashen:phol-cashen/fix-peer-dependecy-and-target-codegen

Conversation

@phillip-cashen

@phillip-cashen phillip-cashen commented Jul 15, 2026

Copy link
Copy Markdown

Hello DA Daml team!

i have a small PR here that i hope you would take a look that addresses two minor concerns/problems i saw when using the codegen.

this PR fixes two areas in the javascript/typescript code generation.

  1. its called "peerDependencies" and not "peer-dependencies" (https://docs.npmjs.com/cli/v7/configuring-npm/package-json#peerdependencies)
  2. "es5" has end-of-life with typescript 6.0, so updating to es2015 makes sense, epseically since it is already included in the "lib" section anyway.

@phillip-cashen phillip-cashen changed the title fix: peer-dependencies and targe for codegen fix: peer-dependencies and target for JS codegen Jul 16, 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