Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

My introduction to the community #86

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Conversation

eddybruv
Copy link

@elhmn please review. I'll appreciate feedback if any.

@eddybruv eddybruv changed the title My introduction to the community My introduction to the community, addresses issue #16 Apr 25, 2022
@eddybruv eddybruv changed the title My introduction to the community, addresses issue #16 My introduction to the community Apr 25, 2022
Copy link
Member

@elhmn elhmn left a comment

Choose a reason for hiding this comment

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

Hey @eddybruv , thank you for this PR. Your changes you made on the PR should only be related to what the issue you are working on.

I would suggest you remove the changes you made on the README.md file.

content/post/contributors/eddybruv.md Outdated Show resolved Hide resolved

I'm happy to belong to this tech community✊.

Nice to meet you💖.
Copy link
Member

Choose a reason for hiding this comment

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

Nice to meet you. It is a pleasure to have you onboard 😄

@eddybruv
Copy link
Author

Hey @eddybruv , thank you for this PR. Your changes you made on the PR should only be related to what the issue you are working on.

I would suggest you remove the changes you made on the README.md file.

Oh, I will.
I didn't notice that change will be added to the pull request since I made the pull request before changing the README.md file.

@eddybruv eddybruv closed this May 10, 2022
@eddybruv eddybruv reopened this May 10, 2022
@eddybruv
Copy link
Author

It's pretty messed up(too many commits) but then, the issue is fixed

@eddybruv eddybruv requested a review from elhmn May 11, 2022 20:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants