Re: Find some consensus


Dave Sand
 

I think the following line needs to be changed:

- All CI tests have passed for the Pull Request

It probably should be "All required CI..."

Dave Sand

----- Original message -----
From: Bob Jacobsen <@BobJacobsen>
To: "jmri@jmri-developers.groups.io Notification" <jmri@jmri-developers.groups.io>
Subject: Re: [jmri-developers] Find some consensus
Date: Sunday, August 02, 2020 10:47 AM

I’ve added some proposed changes in PR pabender#17

https://github.com/pabender/JMRI/pull/17

Direct link to proposed changes:
https://github.com/pabender/JMRI/pull/17/files

Page as changed:
https://htmlpreview.github.io/?https://github.com/bobjacobsen/JMRI/PR8865-bis/help/en/html/doc/Technical/gitadmin.shtml

Bob

On Aug 1, 2020, at 7:59 PM, danielb987 <db123@...> wrote:

Please read #8865 and give your comments on it.

Update github administration page to reflect current discussion
https://github.com/JMRI/JMRI/pull/8865

Bob Jacobsen
@BobJacobsen

Join jmri@jmri-developers.groups.io to automatically receive all group messages.