search

matrix-org / matrix-spec

The Matrix protocol specification
Apache License 2.0
171 stars 91 forks source link

Ship Typescript types for Matrix APIs in @matrix-org/spec #1766

Closed t3chguy closed 3 months ago

t3chguy commented 3 months ago

Example consumer: https://github.com/matrix-org/matrix-js-sdk/pull/3653

Preview: https://pr1766--matrix-spec-previews.netlify.app

t3chguy commented 3 months ago

you can bet that at least one of your target platforms will break

The npm releases have been broken for the last N spec releases anyway, I've been handling them manually

I feel like this should be in a separate repo

Happy to move it but I won't be maintaining multiple npm packages so I'd be taking the current package name with me

richvdh commented 3 months ago

Happy to move it but I won't be maintaining multiple npm packages so I'd be taking the current package name with me

Yup, I (and other members of the SCT) think it makes sense to have a single npm package. Presumably you'd have to pull sas-emoji.json over HTTP from github during the build process, or something?

Sounds good, anyway. If you'd like to make this happen, it would be great!

t3chguy commented 3 months ago

Presumably you'd have to pull sas-emoji.json over HTTP from github during the build process, or something?

Yup, it'd also have to be manual given you can't trigger off release events on other repos.

t3chguy commented 3 months ago

Replaced by https://github.com/t3chguy/matrix-spec-npm