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

July notes: fix links #122

Merged
merged 2 commits into from
Jul 21, 2024
Merged

July notes: fix links #122

merged 2 commits into from
Jul 21, 2024

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Jul 20, 2024

Copy link
Contributor

@egeakman egeakman left a comment

Choose a reason for hiding this comment

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

Thanks Hugo!

docs/monthly-meeting/2024-07.md Outdated Show resolved Hide resolved
@@ -70,8 +70,8 @@ Please take a second to read through it!
- [Ned] interested in the idea spreadsheet; maybe it can help even offline
- [Carol] At last PyCon US, Shauna and Melanie tried to make their first docs
PR, and found some issues in the onboarding process
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
PR, and found some issues in the onboarding process
PR, found some issues in the onboarding process, and documented them
in the following two Google Docs:

This optionally goes with the next suggestion, in case we want to "warn" the users more explicitly that the links points to Google Docs.

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, the other change on its own is a good improvement.

Co-authored-by: Ezio Melotti <[email protected]>
@hugovk hugovk merged commit f843511 into main Jul 21, 2024
6 checks passed
@hugovk hugovk deleted the july-links branch July 21, 2024 07:16
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