Skip to content

Yay the challenge is complete #38

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 2, 2018
Merged

Yay the challenge is complete #38

merged 1 commit into from
Mar 2, 2018

Conversation

claalmve
Copy link

@claalmve claalmve commented Mar 2, 2018

Any comments, discussion, advice, code snippets, etc. are welcome! Thanks for reading :)

Woohoo

stella-lu
stella-lu reviewed Mar 2, 2018
View reviewed changes
day100/hi.txt
@@ -0,0 +1 @@
Hurray it's been 100 days!
Copy link

Choose a reason for hiding this comment

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

has it really?

JinsuElhance
JinsuElhance reviewed Mar 2, 2018
View reviewed changes
Copy link

@JinsuElhance JinsuElhance left a comment

Choose a reason for hiding this comment

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

No lie calvin, this is some shit

thatguyintech
thatguyintech requested changes Mar 2, 2018
View reviewed changes
Copy link
Owner

@thatguyintech thatguyintech left a comment

Choose a reason for hiding this comment

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

fake news

day100/hi.txt
@@ -0,0 +1 @@
Hurray it's been 100 days!
Copy link
Owner

Choose a reason for hiding this comment

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

asfwpef

JinsuElhance
JinsuElhance suggested changes Mar 2, 2018
View reviewed changes
Copy link

@JinsuElhance JinsuElhance left a comment

Choose a reason for hiding this comment

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

I think it needs more emojis

stella-lu
stella-lu suggested changes Mar 2, 2018
View reviewed changes
Copy link

@stella-lu stella-lu left a comment

Choose a reason for hiding this comment

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

w r o n g

@orrblue
Copy link

orrblue commented Mar 2, 2018

I think it's perf

claalmve
claalmve commented Mar 2, 2018
View reviewed changes
Copy link
Author

@claalmve claalmve left a comment •
edited
Loading

Choose a reason for hiding this comment

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

i love this code wow

orrblue
orrblue approved these changes Mar 2, 2018
View reviewed changes
Copy link

@orrblue orrblue left a comment

Choose a reason for hiding this comment

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

I APPROVE

@thatguyintech
Copy link
Owner

Fix #46

@thatguyintech
Copy link
Owner

Redundant work as #45

@thatguyintech thatguyintech merged commit d785750 into thatguyintech:master Mar 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thatguyintech thatguyintech thatguyintech approved these changes

+3 more reviewers

@JinsuElhance JinsuElhance JinsuElhance requested changes

@stella-lu stella-lu stella-lu requested changes

@orrblue orrblue orrblue approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@claalmve @orrblue @thatguyintech @JinsuElhance @stella-lu