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

READ-FIRST: USE GERRITHUB.IO FOR CONTRIBUTIONS #354

Open
wants to merge 1 commit into
base: EOL
Choose a base branch
from

Conversation

queria
Copy link
Collaborator

@queria queria commented Mar 7, 2019

Read https://infrared.readthedocs.io/en/latest/contribute.html
before opening any Pull Requests.

TLDR:

  • Red Hatters should send patches via gerrit (review.gerrithub.io)
    if You are one, DO NOT OPEN PULL REQUESTS please

  • in other cases pull requests are OK
    (You will get bit more of human evaluation)

Change-Id: I8df121bf44ea203b3857de6b5f3451c574a37df7

Read https://infrared.readthedocs.io/en/latest/contribute.html
before opening any Pull Requests.

TLDR:

- Red Hatters should send patches via gerrit (review.gerrithub.io)
  if You are one, DO NOT OPEN PULL REQUESTS please

- in other cases pull requests are OK
  (You will get bit more of human evaluation)

Change-Id: I8df121bf44ea203b3857de6b5f3451c574a37df7
Copy link
Collaborator

@rhosqeauto rhosqeauto left a comment

Choose a reason for hiding this comment

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

DO NOT CLOSE/MERGE

this change is supposed to stay open (and ideally on top of PR list)

@hobbestigrou
Copy link
Contributor

Sorry I have maybe a stupid question, but why the project I have two different ways to contribute? Gerrithub is open and use github. To add a new contributor on the gerrithub instance of the project, it's to add the new contributor in the rhosqeauto-contrib group. No specific internal dependency. I think this patch watch, that not a easy to have two different ways to contribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants