Skip to content
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

Add GitHub reply to the issue if not security. #182

Merged
merged 1 commit into from
Aug 16, 2015

Conversation

cirosantilli
Copy link
Collaborator

No description provided.

@patcon
Copy link

patcon commented Apr 29, 2014

Maybe dup? #94

@cirosantilli
Copy link
Collaborator Author

I had I quick look and seems not to me: this is a CONTRIBUGING-like guideline with a patch for this repo, #94 proposes a computational way of automating things? But maybe I got it wrong. Feel free to close if that is the case.

@patcon
Copy link

patcon commented Apr 29, 2014

Ah... sorry, I didn't notice it was a PR. My bad :)

(And I'm not a repo maintainer, so I can't merge.)

@daveloyall
Copy link

👍 I support the proposed change to README.md. I will start copy/pasting replies from support@github.com as the proposed change suggests.

@jakesylvestre
Copy link
Contributor

As will I

@Gaelan
Copy link

Gaelan commented Jun 24, 2015

We should also always give GH the issue number here when we submit suggestions.

cirosantilli added a commit that referenced this pull request Aug 16, 2015
Add GitHub reply to the issue if not security.
@cirosantilli cirosantilli merged commit df5e197 into isaacs:master Aug 16, 2015
@cirosantilli
Copy link
Collaborator Author

@Gaelan agreed, I always do that. Please send another PR.

@cirosantilli
Copy link
Collaborator Author

@Gaelan ok did it at: #445

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.

6 participants