Skip to content
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

Merge Community meeting #1 Notes #413

Merged
merged 4 commits into from
Nov 11, 2021
Merged

Merge Community meeting #1 Notes #413

merged 4 commits into from
Nov 11, 2021

Conversation

s-wuest
Copy link
Contributor

@s-wuest s-wuest commented Nov 7, 2021

Documentation of notes of Merge Community meeting #1 according to
https://github.com/ethereum-cat-herders/PM/issues/246

@s-wuest
Copy link
Contributor Author

s-wuest commented Nov 7, 2021

@poojaranjan Done. Pls review. Best regards!!

@poojaranjan
Copy link
Contributor

poojaranjan commented Nov 8, 2021

Thanks for documenting the meeting notes, @stefan-wuest!

It would be nice to have

  • add a "Summary section" (Decisions Made / Action items) to capture any important points of the general agenda item
  • remove "um" and "uh"
  • fix punctuations
  • break it into sentences to provide a better understanding of the context (Ref line 38, 219)

@s-wuest
Copy link
Contributor Author

s-wuest commented Nov 8, 2021

Thanks for the review @poojaranjan

I will take the above points into account and adapt the notes accordingly by tomorrow night.

s-wuest and others added 3 commits November 9, 2021 18:29
Following updates according to input by @poojaranjan

- Added "Summary section" (Decisions Made / Action items)  -> It was difficult for me to detect what matters here so pls add if somethings missing.
- removed "um" and "uh"
- fixed punctuations
- broke it into sentences to provide a better understanding of the context
@s-wuest
Copy link
Contributor Author

s-wuest commented Nov 10, 2021

Hi @poojaranjan
I updated it according to your input. Pls review again. Thx.

@timbeiko timbeiko merged commit e50aca9 into ethereum:master Nov 11, 2021
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.

3 participants