Skip to content

Conversation

@MaxDesiatov
Copy link
Member

@MaxDesiatov MaxDesiatov commented Oct 18, 2020

I hope I didn't miss anything, suggestions and feedback are welcome as usual.

I've also added new fonts to the theme so that it feels less bland.

Before:

Screenshot 2020-10-29 at 22 09 38

After:

Screenshot 2020-10-29 at 22 09 50

@MaxDesiatov MaxDesiatov requested a review from a team October 29, 2020 21:51
@MaxDesiatov MaxDesiatov marked this pull request as ready for review October 29, 2020 21:51
MaxDesiatov and others added 3 commits October 30, 2020 07:53
Co-authored-by: Yuta Saito <kateinoigakukun@gmail.com>
Co-authored-by: Yuta Saito <kateinoigakukun@gmail.com>
Co-authored-by: yonihemi <jonathan@hemi.dev>
@MaxDesiatov MaxDesiatov merged commit f5bce90 into main Oct 30, 2020
@MaxDesiatov MaxDesiatov deleted the update-03 branch October 30, 2020 09:09
support. This mirrors our existing [C++ implementation in the legacy
driver](https://github.com/swiftwasm/swift/blob/swiftwasm/lib/Driver/WebAssemblyToolChains.cpp), and
it's great that the new parts of the Swift compiler rewritten in Swift are going to support
WebAssembly too. While Swift Driver in that enabled in any toolchain by default, we're definitely
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not quite sure what the beginning of this sentence is saying. Maybe:

Suggested change
WebAssembly too. While Swift Driver in that enabled in any toolchain by default, we're definitely
WebAssembly too. While Swift Driver isn't enabled in any toolchain by default, we're definitely

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, that's updated in d028498.

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.

5 participants