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

Update index.md #27303

Closed
wants to merge 1 commit into from
Closed

Update index.md #27303

wants to merge 1 commit into from

Conversation

Sgk3217
Copy link
Contributor

@Sgk3217 Sgk3217 commented Oct 23, 2018

I just wanted to add how people are the most vulnerable when it comes to security

  • [x ] I have read freeCodeCamp's contribution guidelines.
  • [ x] My pull request has a descriptive title (not a vague title like Update index.md)
  • [ x] My pull request targets the master branch of freeCodeCamp.
  • [ x] None of my changes are plagiarized from another source without proper attribution.
  • [ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.

Closes #XXXXX

I just wanted to add how  people are the most vulnerable when it comes to security
@ArielLeslie
Copy link
Contributor

You ignored a list of requirements including "My pull request has a descriptive title (not a vague title like Update index.md)". Descriptive titles and good explanations of changes are necessary for a well maintained codebase. Free Code Camp is choosing to relax some of its requirements to help people reach their Hacktoberfest goals, but this PR would normally be rejected. In the future, please help repository maintainers by providing useful titles and do not ignore contributing guidelines or requirements.

@RandellDawson
Copy link
Member

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

@thecodingaviator
Copy link
Contributor

Thank you for opening this pull request.

This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.

Thank you and happy coding. ✈️

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.

5 participants