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

[READY FOR REVIEW] may + june coworking notes #3662

Merged
merged 10 commits into from
Jul 15, 2024
Merged

Conversation

aleesteele
Copy link
Member

Summary

Notes from may coworking calls ready for review

Copy link

netlify bot commented May 28, 2024

Deploy Preview for the-turing-way ready!

Name Link
🔨 Latest commit 2923ec3
🔍 Latest deploy log https://app.netlify.com/sites/the-turing-way/deploys/66958ee504d497000834a60b
😎 Deploy Preview https://deploy-preview-3662--the-turing-way.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RichardJActon
Copy link
Collaborator

Hi @aleesteele Is there a convention of someone else who attended the meetings minuted in these notes approving the merge or can anyone do it?

@aleesteele aleesteele changed the title [READY FOR REVIEW] may coworking notes [READY FOR REVIEW] may + june coworking notes Jun 17, 2024
@aleesteele
Copy link
Member Author

@RichardJActon Thanks for your question Richard!

We don't have an established process (yet), but as general practice so far we have had delivery staff team review these PRs to double check any information that should not be archived here. There can of course be multiple reviewers to check for typos, formatting, etc! 👍

@RichardJActon
Copy link
Collaborator

Thanks @aleesteele good to know - was just checking through any PRs I might be able to give a quick review after book dash.

Copy link
Collaborator

@Arielle-Bennett Arielle-Bennett left a comment

Choose a reason for hiding this comment

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

Minor comments and a couple of typoes corrected :)

governance/community-calls/20240610-coworking.md Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a duplicte and/or is missing notes - do we want to add something around them intentionally being left blank if there are no notes to add?

governance/community-calls/20240617-coworking.md Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Missing notes? Worth deliberately noting "no notes taken"?

aleesteele and others added 4 commits June 28, 2024 10:08
Co-authored-by: Arielle-Bennett <74651964+Arielle-Bennett@users.noreply.github.com>
Co-authored-by: Arielle-Bennett <74651964+Arielle-Bennett@users.noreply.github.com>
@aleesteele
Copy link
Member Author

Thanks a lot for the review, @Arielle-Bennett! Fixed the missing/duplicate notes, and added in missing notes from July :)

@aleesteele aleesteele merged commit 5058ae7 into main Jul 15, 2024
3 checks passed
@aleesteele aleesteele deleted the coworking-may24 branch July 15, 2024 21:07
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.

None yet

3 participants