Skip to content

Project 2 (Overflow 1)#5

Open
nkrishnan19 wants to merge 1 commit intomainfrom
branch2
Open

Project 2 (Overflow 1)#5
nkrishnan19 wants to merge 1 commit intomainfrom
branch2

Conversation

@nkrishnan19
Copy link
Contributor

@nkrishnan19 nkrishnan19 commented Jan 5, 2023

Summary

In this PR, I have included my contribution to be reviewed for Project 2!

Motivation

By creating this PR, students can review, approve, and link this PR to their issue.

Testing

No testing required

@nkrishnan19 nkrishnan19 changed the title new file Project 2 (Overflow 1) Jan 5, 2023
@sagars729 sagars729 self-requested a review January 5, 2023 14:33
@sagars729 sagars729 linked an issue Jan 5, 2023 that may be closed by this pull request
Sagar Saxena - My First Card #11
Open
sagars729
sagars729 approved these changes Jan 5, 2023
View reviewed changes
Copy link
Collaborator

@sagars729 sagars729 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@shadow-smoke shadow-smoke self-requested a review January 5, 2023 20:40
@ghost ghost self-requested a review January 5, 2023 20:42
shadow-smoke
shadow-smoke approved these changes Jan 5, 2023
View reviewed changes
Copy link

@shadow-smoke shadow-smoke left a comment

Choose a reason for hiding this comment

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

Reviewed and completed

ghost
ghost approved these changes Jan 5, 2023
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Looking good

@shadow-smoke shadow-smoke linked an issue Jan 5, 2023 that may be closed by this pull request
Tyler Jenkins - First Card #16
Open
@ghost ghost linked an issue Jan 5, 2023 that may be closed by this pull request
Justin Pratama - First Card #17
Open
@DanXu007 DanXu007 self-requested a review January 6, 2023 01:27
DanXu007
DanXu007 approved these changes Jan 6, 2023
View reviewed changes
Copy link

@DanXu007 DanXu007 left a comment

Choose a reason for hiding this comment

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

Leaving a comment here!

@DanXu007 DanXu007 linked an issue Jan 6, 2023 that may be closed by this pull request
Dan Xu - First Card #22
Open
@DanXu007 DanXu007 removed a link to an issue Jan 6, 2023
Dan Xu - First Card #22
Open
This was linked to issues Jan 6, 2023
Dan Xu - First Card #22
Open
Dan Xu - First Card #23
Open
@fatima-0000 fatima-0000 self-requested a review January 6, 2023 06:02
fatima-0000
fatima-0000 approved these changes Jan 6, 2023
View reviewed changes
Copy link

@fatima-0000 fatima-0000 left a comment

Choose a reason for hiding this comment

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

Approving the changes!

@fatima-0000 fatima-0000 linked an issue Jan 6, 2023 that may be closed by this pull request
Fatima Chariwala - First Card #30
Open
@nachado12 nachado12 self-requested a review January 6, 2023 20:45
nachado12
nachado12 approved these changes Jan 6, 2023
View reviewed changes
Copy link

@nachado12 nachado12 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@nachado12 nachado12 linked an issue Jan 6, 2023 that may be closed by this pull request
Neal Machado - First Card #36
Open
@nhajela1 nhajela1 self-requested a review January 7, 2023 03:17
nhajela1
nhajela1 approved these changes Jan 7, 2023
View reviewed changes
Copy link

@nhajela1 nhajela1 left a comment

Choose a reason for hiding this comment

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

Looks good!

@nhajela1 nhajela1 linked an issue Jan 7, 2023 that may be closed by this pull request
Neil Hajela - First Card #43
Open
mehulbehera
mehulbehera approved these changes Jan 10, 2023
View reviewed changes
Copy link

@mehulbehera mehulbehera left a comment

Choose a reason for hiding this comment

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

Looks good!

@arshiv27 arshiv27 self-requested a review January 10, 2023 04:56
@zaidisyed1991 zaidisyed1991 requested review from zaidisyed1991 and removed request for zaidisyed1991 January 12, 2023 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sagars729 sagars729 sagars729 approved these changes

@nachado12 nachado12 nachado12 approved these changes

@nhajela1 nhajela1 nhajela1 approved these changes

@shadow-smoke shadow-smoke shadow-smoke approved these changes

@mehulbehera mehulbehera mehulbehera approved these changes

@fatima-0000 fatima-0000 fatima-0000 approved these changes

@DanXu007 DanXu007 DanXu007 approved these changes

@arshiv27 arshiv27 Awaiting requested review from arshiv27

+1 more reviewer
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

8 participants

@nkrishnan19 @sagars729 @nachado12 @nhajela1 @shadow-smoke @mehulbehera @fatima-0000 @DanXu007