Skip to content

Edits for clarity - #1110

Merged
sei-renae merged 3 commits into
copilot/add-adr-for-terminology-shiftfrom
ahouseholder-patch-1
Apr 3, 2026
Merged

sei-renae merged 3 commits into
copilot/add-adr-for-terminology-shiftfrom
ahouseholder-patch-1

Conversation

@ahouseholder

@ahouseholder ahouseholder commented Apr 2, 2026 •

Copy link
Copy Markdown
Contributor

Refining the wording in places for accuracy & clarity.

Also removes emphasis formatting from the "Considered Options" list items in ADR-0014 to match the plain-text style used consistently across other ADRs in the repository.

Refining the wording in places for accuracy & clarity.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR refines wording in ADR-0014 to clarify why SSVC is standardizing on “decision table” terminology and to better distinguish operations-research vs. machine-learning meanings of “decision tree”.

Changes:

  • Reworded context/problem statement to explicitly address terminology conflicts and add references.
  • Clarified decision drivers/options and expanded rationale with “decision model” as a broader term.
  • Tweaked confirmation and added additional “More Information” links.

Comment thread docs/adr/0014-use-decision-table-terminology.md Outdated
Comment thread docs/adr/0014-use-decision-table-terminology.md Outdated
Comment on lines +36 to +38
- Switch to *decision framework*
- Switch to *decision table*
- Switch to *decision model*

Copilot AI Apr 2, 2026

Copy link

Choose a reason for hiding this comment

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

The “Considered Options” list uses emphasis for the option names. Other ADRs in this repo list options as plain text (e.g., docs/adr/0002-ssvc-decision-points-are-versioned.md:29-32, docs/adr/0007-descriptions-exclude-examples.md:28-29). Consider keeping these un-emphasized for consistency and easier scanning/searching.

Copilot uses AI. Check for mistakes.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Done — removed the emphasis from the option names in the "Considered Options" list so they match the plain-text style used in other ADRs. (commit 25b68d8)

Comment thread docs/adr/0014-use-decision-table-terminology.md Outdated
Comment thread docs/adr/0014-use-decision-table-terminology.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sei-renae
sei-renae merged commit c48f706 into copilot/add-adr-for-terminology-shift Apr 3, 2026
1 check passed
@sei-renae
sei-renae deleted the ahouseholder-patch-1 branch April 3, 2026 17:59
@ahouseholder ahouseholder added this to the 2026-03 milestone Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants