Skip to content

Comment on Binaryen proposal #91

Description

@ericvergnaud

Hi there,

to avoid observed gaps between the TypeScript in this repo and the underlying JavaScript in Binaryen repo, I have submitted a PR that moves the TypeScript source to the Binaryen repo and generates the JavaScript from it, thus ensuring a single source of truth.

The PR is here WebAssembly/binaryen#6192. (If approved, that PR would be followed by a corresponding PR in this repo, adjusting for the change of source of truth for TypeScript)

It would be great if you could comment on that PR rapidly such that the Binaryen team can reach a decision on whether or not to accept the PR.

Activity

  1. dcodeIO commented on Jan 14, 2024

    @dcodeIO
    Member

    I do like the idea of replacing the JS plus separate typings with TS plus generated typings. I'd probably go even further and also generate the API docs from tsc's type info and respective comments while on it. Or even further, and generate all of it while also guaranteeing that the C++, C and JS APIs are in sync.

  2. ericvergnaud commented on Jan 14, 2024

    @ericvergnaud
    ContributorAuthor

    Following a counter PR WebAssembly/binaryen#6224, I've opened a discussion here: WebAssembly/binaryen#6225

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions