Skip to content
This repository was archived by the owner on Apr 25, 2025. It is now read-only.

Few style fixes#232

Merged
aheejin merged 2 commits intoWebAssembly:mainfrom
aheejin:rephrase
Oct 8, 2022
Merged

Few style fixes#232
aheejin merged 2 commits intoWebAssembly:mainfrom
aheejin:rephrase

Conversation

@aheejin
Copy link
Member

@aheejin aheejin commented Sep 12, 2022

- Change 'We denote X = Y' to 'Let A be Y' (suggested in
  WebAssembly#180 (comment))
- Remove uses of 'we' pronouns (suggested in
  WebAssembly#180 (comment))
@aheejin aheejin requested review from ioannad and rossberg September 12, 2022 04:50
Copy link
Collaborator

@ioannad ioannad left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Co-authored-by: Andreas Rossberg <rossberg@mpi-sws.org>
Co-authored-by: Ioanna M Dimitriou H <9728696+ioannad@users.noreply.github.com>
@ioannad
Copy link
Collaborator

ioannad commented Oct 7, 2022

Should I merge this @aheejin or would you rather do the merge?

@aheejin
Copy link
Member Author

aheejin commented Oct 8, 2022

I was waiting for #219 to land because the two have merge conflicts, but I guess I can land first, if you fix the conflicts 😅

@aheejin aheejin merged commit 2429b1f into WebAssembly:main Oct 8, 2022
@aheejin aheejin deleted the rephrase branch October 8, 2022 03:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants