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

Adds zo0o0ot #1263

Merged
merged 4 commits into from
Oct 6, 2024
Merged

Adds zo0o0ot #1263

merged 4 commits into from
Oct 6, 2024

Conversation

zo0o0ot
Copy link
Contributor

@zo0o0ot zo0o0ot commented Oct 6, 2024

Linked Issue

#13

Description

This pull request adds zo0o0ot to the members list.

Methodology

I followed the directions in #13 to make sure I was doing the right work. I'm also mentoring @pkcnj and I wanted to know the work that they are doing.
Note: I've been participating in Hacktoberfest since 2015, but only since 2023 with VirtualCoffee, so I'm not sure if I'm supposed to have the 2022 badge.

Code of Conduct

By submitting this pull request, you agree to follow our Code of Conduct
Agreed!

@zo0o0ot zo0o0ot requested a review from a team as a code owner October 6, 2024 03:45
Copy link

netlify bot commented Oct 6, 2024

👷 Deploy Preview for virtual-coffee-io processing.

Name Link
🔨 Latest commit c5f5320
🔍 Latest deploy log https://app.netlify.com/sites/virtual-coffee-io/deploys/67020d363db01900082720ba

@zo0o0ot
Copy link
Contributor Author

zo0o0ot commented Oct 6, 2024

Note: I'm realizing that I had bad branch etiquette, and I did this in my main branch instead of members/zo0o0ot. Sorry about that, and let me know if I should close my PR and try again with that.

@@ -2,7 +2,7 @@ import type { MemberObject } from '../types';
// import { profileMasks } from '../flare';

// Change _EXAMPLE to `yourGitHubUserName` and add your info below
export const _EXAMPLE: MemberObject = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I found this error after committing the same error myself. It showed up in the logs.

Copy link
Contributor

Choose a reason for hiding this comment

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

praise: Good catch! Thanks, Ross!

Copy link
Contributor

@meg-gutshall meg-gutshall left a comment

Choose a reason for hiding this comment

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

@@ -2,7 +2,7 @@ import type { MemberObject } from '../types';
// import { profileMasks } from '../flare';

// Change _EXAMPLE to `yourGitHubUserName` and add your info below
export const _EXAMPLE: MemberObject = {
Copy link
Contributor

Choose a reason for hiding this comment

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

praise: Good catch! Thanks, Ross!

@meg-gutshall meg-gutshall merged commit 4ef8adc into Virtual-Coffee:main Oct 6, 2024
5 checks passed
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.

2 participants