Skip to content

Bump version to 0.14.0, update CHANGELOG.md #181

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 8, 2022
Merged

Bump version to 0.14.0, update CHANGELOG.md #181

merged 1 commit into from
Apr 8, 2022

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Apr 8, 2022

New release is required to resolve TokamakUI/Tokamak#475.

I've also cleaned up formatting in .ts file and clarified version incompatibility error message.

@MaxDesiatov MaxDesiatov requested a review from a team April 8, 2022 08:39
@MaxDesiatov MaxDesiatov added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Apr 8, 2022
@MaxDesiatov MaxDesiatov enabled auto-merge (squash) April 8, 2022 08:40
@MaxDesiatov MaxDesiatov merged commit 9a3b7d5 into main Apr 8, 2022
@MaxDesiatov MaxDesiatov deleted the v0.14.0 branch April 8, 2022 08:42
@github-actions
Copy link

github-actions bot commented Apr 8, 2022

Time Change: +1,152.5ms (10%) ⚠️

Total Time: 11,165ms

Test name Duration Change
Serialization/Swift Int to JavaScript 3,350.5ms +277.25ms (8%) 🔍
Serialization/Swift String to JavaScript 3,923.75ms +893.25ms (22%) 🚨
ℹ️ View Unchanged
Test name Duration Change
Serialization/Write JavaScript number directly 220ms +2.5ms (1%)
Serialization/Write JavaScript string directly 221ms 0ms
Object heap/Increment and decrement RC 3,449.75ms -20.5ms (0%)

performance-action

@kateinoigakukun
Copy link
Member

kateinoigakukun commented Apr 8, 2022

BTW I think this new release is not required for the Tokamak PR. Is there any essential change?

@MaxDesiatov
Copy link
Contributor Author

MaxDesiatov commented Apr 8, 2022

swiftwasm/carton#310 relies on reactor model, which was updated in #180, wasn't it? And then .jsValue() -> .jsValue change was breaking, I'd like to update Tokamak codebase for it sooner rather than later.

Either way, I've already tagged it, sorry if that was premature 😅

@kateinoigakukun
Copy link
Member

#180 is only for test harnesses, so no change in the js runtime. Yeah, it's fine to release breaking changes soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants