cedar-policy / cedar

Implementation of the Cedar Policy Language
https://www.cedarpolicy.com
Apache License 2.0
889 stars 80 forks source link

Workaround wasm build error #1288

Closed john-h-kastner-aws closed 3 weeks ago

john-h-kastner-aws commented 4 weeks ago

Description of changes

Workaround https://github.com/rustwasm/wasm-bindgen/issues/4207 by applying fix proposed in https://github.com/rustwasm/wasm-bindgen/pull/4210 with a sed script.

This fix is a bit hacky, so we might prefer leaving the WASM build broken until this gets fixed in wasm-bindgen.

Issue #, if available

Checklist for requesting a review

The change in this PR is (choose one, and delete the other options):

I confirm that this PR (choose one, and delete the other options):

I confirm that cedar-spec (choose one, and delete the other options):

I confirm that docs.cedarpolicy.com (choose one, and delete the other options):

john-h-kastner-aws commented 3 weeks ago

I've decided I like #1291 better, and it seems to pass CI, so I'll close this