Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Add CONTRIBUTING.md & CODE_OF_CONDUCT.md #925

Merged
merged 5 commits into from
Dec 4, 2019
Merged

Conversation

wadells
Copy link
Contributor

@wadells wadells commented Dec 4, 2019

Resolves #923 by adding the code of conduct, and takes a stab at contribution guidelines. CONTRIBUTING felt appropriate to give discoverability to this info from the README.

Testing Done:

View these three files in my branch:

https://github.com/wadells/gravity/tree/code-of-conduct#contributing
https://github.com/wadells/gravity/blob/code-of-conduct/CONTRIBUTING.md
https://github.com/wadells/gravity/blob/code-of-conduct/CODE_OF_CONDUCT.md

Double checked that the links work.

@klizhentas
Copy link
Contributor

@r0mant @knisbet take a look seems like Jenkins is not happy, but should be fine to merge as it's doc only?

@wadells
Copy link
Contributor Author

wadells commented Dec 4, 2019

#924 failed in jenkins because of https://github.com/gravitational/robotest/issues/154 (which I have an idea of how to fix), and this PR is inheriting the build status because the commit hash is identical. I retriggered, but it is hitting some sort of GH API throttling:

01:29:31 GitHub API Usage: Current quota has 3051 remaining (604 over budget). Next quota of 5000 in 54 min. Sleeping for 10 min.
01:32:31 GitHub API Usage: The quota may have been refreshed earlier than expected, rechecking...
01:32:31 GitHub API Usage: Current quota has 3051 remaining (416 over budget). Next quota of 5000 in 51 min. Sleeping for 7 min 4 sec.

01:35:31 GitHub API Usage: The quota may have been refreshed earlier than expected, rechecking...
01:35:31 GitHub API Usage: Current quota has 3051 remaining (228 over budget). Next quota of 5000 in 48 min. Sleeping for 4 min 4 sec.

As seen here:

https://jenkins.gravitational.io/job/Gravity-PR-5.2/job/PR-924/3/console

Once that clears up, I'd expect the CI to be green.

Once you know you have a issue, make sure to fill out all sections of the
one of the templates at https://github.com/gravitational/gravity/issues/new/choose.

Gravity contributors will triage the issue shortly.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is it safe to make this promise? I don't know what our issues/triage policy is currently.

CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@r0mant r0mant left a comment

Choose a reason for hiding this comment

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

Just a couple of typos.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
While reviewing onboarding materials, @klizhentas noticed that the
gravity repo was missing the code of conduct that was present in
teleport. For consistency, adding it here.

Addresses #923.
As I was looking at #923 (Gravity missing a code of conduct) I realised
it didn't have any contributing info. Adding a CONTRIBUTING may help
foster outsite engagement.

Inspiration drawn from Teleport's CONTRIBUTING, mashed up with golang's.
This will help with discoverability, and tie the docs together more
seamlessly.
Gravity uses the latter, as mentioned in:
   #925 (comment)
@wadells wadells merged commit 41e9a6a into master Dec 4, 2019
@wadells wadells deleted the walt/code-of-conduct branch March 11, 2020 20:27
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.

[BUG] Missing code of conduct
4 participants