Skip to content

docs: Add all the readmes - #23

Open
ldiego08 wants to merge 4 commits into
mainfrom
docs/readmes
Open

docs: Add all the readmes#23
ldiego08 wants to merge 4 commits into
mainfrom
docs/readmes

Conversation

@ldiego08

Copy link
Copy Markdown
Contributor

Proposed changes

  • Add CODE_OF_CONDUCT, CONTRIBUTORS, and SECURITY.
  • Add Github issue and PR templates

maccath
maccath previously approved these changes Oct 14, 2021

@maccath maccath left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One grammar issue, but OK to merge. :)

Comment thread CONTRIBUTING.md Outdated
Co-authored-by: Katy Ereira <katy@skillshare.com>
jeffreybaird
jeffreybaird previously approved these changes Oct 14, 2021
ramsey
ramsey previously approved these changes Oct 14, 2021

@ramsey ramsey left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great. We need to figure out what email address to add to the code of conduct.

Comment thread CODE_OF_CONDUCT.md Outdated
## Enforcement

Instances of abusive, harassing, or otherwise unacceptable behavior may be
reported by contacting the project team at {{ email }}. All

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We need to put an email address here for {{ email }}

@ldiego08
ldiego08 dismissed stale reviews from ramsey and jeffreybaird via 0280c22 October 14, 2021 17:34
@jrode

jrode commented Oct 18, 2021

Copy link
Copy Markdown

Looks great. I just created apollo-federation-php@skillshare.com email group -- @ldiego08 can you switch the email references in this PR to use that address?

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