Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update README.md #25

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

Closed
Sparta-9 wants to merge 1 commit into epicweb-dev:main from Sparta-9:main
Closed

Update README.md #25

Sparta-9 wants to merge 1 commit into epicweb-dev:main from Sparta-9:main

Conversation

@Sparta-9
Copy link

@Sparta-9 Sparta-9 commented Oct 1, 2020

No description provided.

Copy link

@thehairy thehairy left a comment

Choose a reason for hiding this comment

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

Useless as nearly every PR in the last 12 hours.

Copy link

Useless as nearly every PR in the last 12 hours.

Anyone trying to help and improve is always welcomed.


- `src/exercise/00.md`: Background, Exercise Instructions, Extra Credit
- `src/exercise/00.js`: Exercise with Emoji helpers
- `src/exercise/00.js`: Exercise with Emoji helper's
Copy link

@JacobMGEvans JacobMGEvans Oct 1, 2020

Choose a reason for hiding this comment

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

Grammatically I don't think this helper has possession of anything.

Copy link

thehairy commented Oct 1, 2020

Useless as nearly every PR in the last 12 hours.

Anyone trying to help and improve is always welcomed.

Nobody in the last few hours tried to help you. They just want to get as much PR's as possible for Hacktoberfest.
So please don't fall in that trap and just close those spammy PRs.

JacobMGEvans reacted with heart emoji

- **Chuck the Checkered Flag** 🏁 will indicate that you're working with a final
version
- **Marty the Money Bag** 💰 will give you specific tips (and sometimes code)
- **Marty the Money Bag** 💰 will give you a specific tips (and sometimes code)
Copy link

@JacobMGEvans JacobMGEvans Oct 1, 2020

Choose a reason for hiding this comment

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

The grammar without the "a" was correct, "a" would for a singular tip, not plural "tips" 😄

Copy link

can be safely closed @kentcdodds

marcosvega91 reacted with thumbs up emoji

Copy link
Member

🤖 This is Hacktoberfest spam and will be automatically deleted.

If you feel this was in error, please comment.

@kentcdodds kentcdodds added the invalid This doesn't seem right label Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@JacobMGEvans JacobMGEvans JacobMGEvans left review comments

@thehairy thehairy thehairy left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

invalid This doesn't seem right

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /