Elvas Tower: Updated contributing guidelines - Elvas Tower

Jump to content

Posting Rules

All new threads will be started by members of the Open Rails team, Staff, and/or Admins. Existing threads started in other forums may get moved here when it makes sense to do so.

Once a thread is started any member may post replies to it.
Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

Updated contributing guidelines Rate Topic: -----

#1 User is offline   James Ross 

  • Open Rails Developer
  • Group: Status: Elite Member
  • Posts: 5,491
  • Joined: 30-June 10
  • Gender:Not Telling
  • Simulator:Open Rails
  • Country:

Posted 21 January 2022 - 01:47 PM

Having had some time working with the current contributing guidelines and the requirements it sets out, we (ORMT) thought it was time to update this to correct some problems and (hopefully) improve overall clarity.

You can view the pull request or the new page directly

Some highlights of the changes:

  • The documentation and translation exceptions are back - we might have been doing it this way all along but they were somehow missing from the current version
  • The small bug fix exception is back - if the change is <= 10 lines added or removed, and the reviewer agrees it is clear enough of a change, no need for a bug report
  • New features can be created as draft pull requests without anything else - we still require the forum discussion and road-map card (and its approval) before the pull request can be approved, but the order of things is now much more flexible and hopefully much clearer with the diagram I've added
  • The requirement for putting links in the merge commit itself is gone - it was hard and is no longer necessary for triage
Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

1 User(s) are reading this topic
0 members, 1 guests, 0 anonymous users