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

Add extra guidance on mentored pair programming #1133

Merged
merged 12 commits into from
Oct 28, 2024

Conversation

illicitonion
Copy link
Member

What does this change?

We want to specifically call out to volunteers that, unlike in other pair programming, they're not meant to write code or say what code to write.

Org Content?

Pair programming guide

Checklist

We want to specifically call out to volunteers that, unlike in other
pair programming, they're not meant to write code _or_ say what code to
write.
Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-sdc ready!

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-sdc/deploys/671fb89b0119880008df6c19
😎 Deploy Preview https://deploy-preview-1133--cyf-sdc.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.

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-programming ready!

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-programming/deploys/671fb89bbedfa100082b6499
😎 Deploy Preview https://deploy-preview-1133--cyf-programming.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 85 (🔴 down 13 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 90 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-common canceled.

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-common/deploys/671fb89b53679200085b9727

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-tracks ready!

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-tracks/deploys/671fb89b382f3200078a19cd
😎 Deploy Preview https://deploy-preview-1133--cyf-tracks.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.

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-curriculum canceled.

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-curriculum/deploys/671fb89baf66d1000838e6bc

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-piscine ready!

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-piscine/deploys/671fb89b97c1210008ebc938
😎 Deploy Preview https://deploy-preview-1133--cyf-piscine.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (🟢 up 3 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

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

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-launch ready!

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-launch/deploys/671fb89b9e2385000825852b
😎 Deploy Preview https://deploy-preview-1133--cyf-launch.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.

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for cyf-itd canceled.

Name Link
🔨 Latest commit acd207e
🔍 Latest deploy log https://app.netlify.com/sites/cyf-itd/deploys/671fb89bf014650009da9ca3

@moneyinthesky
Copy link
Contributor

Great stuff @illicitonion, thank you 🙌

@illicitonion
Copy link
Member Author

Done - PTAL!

Copy link
Member

@SallyMcGrath SallyMcGrath left a comment

Choose a reason for hiding this comment

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

Love it! Thanks for this!

Maybe some resources too at the bottom?

https://teachtogether.tech/en/#s:classroom-pair
https://www.csteachingtips.org/tips-pair-programming

At some point we should set up a resources taxonomy as a mountable module

org-cyf-guides/content/pair-programming/_index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/_index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/_index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/mentored/index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/mentored/index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/mentored/index.md Outdated Show resolved Hide resolved
org-cyf-guides/content/pair-programming/mentored/index.md Outdated Show resolved Hide resolved
@illicitonion
Copy link
Member Author

I pulled out a separate driver/navigator page, partially for explicit clarity, and partially because those resources are both very driver/navigator specific. PTAL!

Copy link
Member

@SallyMcGrath SallyMcGrath left a comment

Choose a reason for hiding this comment

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

It's great -- really rate this

@illicitonion illicitonion enabled auto-merge (squash) October 28, 2024 16:19
@illicitonion illicitonion merged commit 4b91324 into main Oct 28, 2024
34 checks passed
@illicitonion illicitonion deleted the dwh-mentored-pair-programming branch October 28, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants