You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is the protocol for creating and approving Architectural Design Records for the SasView organization. This process extends to all repositories under the organization.
1. Suggestion Phase
At the appropriate GitHub level (Organization or Repo) create a new discussion under the ADR Suggestions category.
Use a title that describes what the ADR concerns. e.g.: Qt GUI element naming convention. The title may include ADR Suggestion if desired, but is not required.
The ADR should outline what items are required and what items are encouraged. For the purposes of this ADR, all items are required.
ADR suggestions and comments should not be edited for content. Typos and formatting issue edits are permissible, but substantive changes to the ADR should only be included as comments to ensure people new to the discussion can follow the entire course of the process.
Technical meeting participants will review any recent ADRs submissions.
During the Implementation Phase: no more than 3-4 discussion and/or ADRs will be brought to each meeting until all outstanding discussions are resolved.
Technical meeting participants will offer a suggestion on whether to implement the ADR, including any proposed changes, and any implementation phases, if required.
All technical meeting suggestions, whether to approve or deny, will be presented at the bi-weekly meeting for a larger review.
A person will be assigned to write the proposed final ADR wording. Prior to the bi-weekly meeting, this proposal will be added as a comment to the ADR Suggestion for easier review.
All discussion points from meetings will be added as comments to the Discussion by the person leading any discussion on the ADR.
ADRs rejected by both the technical group and biweekly meetings will be closed, as described in heading 3 below and will not proceed beyond this point in the suggestion phase.
The results of the larger review will be sent to the developers mailing list to give developers unable to attend meetings a chance to review the suggestion. This announcement shall include a title making it clear the message is related to ADR approvals, a link to the comment with the proposed ADR, and an anticipated date of final decision coinciding with a bi-weekly meeting at least 3 weeks in the future.
Developers may comment on the proposal in the ADR, during the final meeting, or by email, but all comments must be added to the Discussion by the person leading the ADR discussion.
A final decision on an ADR will be made at the announced meeting by the participants in attendeance and tasks from the following headings 2, 3, and 4 will be assigned to a specific person or people at the time of decision.
2. Accepted ADR
A SasView maintainer or admin will create a new Discussion under the 'ADR' category.
The title should describe the issue the ADR is addressing, and should be similar to title from the Suggestion Phase.
The description of the ADR should be a comprehensive resume of what has been accepted.
The description should include a revision number and link to the previous revision of the ADR, if it exists.
The description should include links to the original discussion(s).
The original discussion(s) should be closed.
3. Rejected ADR
An outline of why the ADR was rejected will be entered into the discussion as a final comment.
The original discussion(s) will be closed.
Appealing a rejection may be made if new information or ideas are brought to either the bi-weekly or techinical meetings and the attendees agree the ADR be revisited. In this case, the person appealing the ADR should reopen the discussion, and include the new information.
Reopening a rejected ADR restarts the process at step 7 of the Suggestion Phase.
4. Tracking Implementation
Once the ADR is accepted and finalized, create corresponding issues in the relevant repository (or repositories) to track the implementation.
The issue body should include a reference to the ADR the issue is based off.
If the ADR impacts multiple projects or components, apply the label Cross Package Issue to each related issues and, once all are created, edit the issue descriptions to include links to all related issues.
5. Revision Process
To revise an ADR start a new discussion as in Suggestion Phase (1), but start the title with the word Revision: and follow it with the ADR title.
The original ADR should be copied/pasted in the new ADR and indented using the Quote option in the editor.
Any changes to the text should use the following styles: Additions should be underlined, Modifications should be in italics, and deletions should be struckout.
The reason(s) for modifying the ADR should be listed as a separate summary outside the quoted section.
The revision process will be the same as the normal process.
As described in Accepted ADR (4), increment the revision number and add a link to the previous revision in the updated ADR.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
This is the protocol for creating and approving Architectural Design Records for the SasView organization. This process extends to all repositories under the organization.
1. Suggestion Phase
ADR Suggestionscategory.Qt GUI element naming convention. The title may includeADR Suggestionif desired, but is not required.eance and tasks from the following headings 2, 3, and 4 will be assigned to a specific person or people at the time of decision.2. Accepted ADR
3. Rejected ADR
4. Tracking Implementation
Cross Package Issueto each related issues and, once all are created, edit the issue descriptions to include links to all related issues.5. Revision Process
Revision:and follow it with the ADR title.Quoteoption in the editor.deletions should be struckout.Revision Acceptance Date: February 10, 2026
Revision Number: 2
Previous Revision: https://github.com/orgs/SasView/discussions/3357
Original Suggestion: https://github.com/orgs/SasView/discussions/3583
Beta Was this translation helpful? Give feedback.
All reactions