Skip to content

docs: Add RATIONALE.md - #200

Merged
philipaconrad merged 1 commit into
open-policy-agent:mainfrom
philipaconrad:philip/rationale-docs
Sep 22, 2026
Merged

philipaconrad merged 1 commit into
open-policy-agent:mainfrom
philipaconrad:philip/rationale-docs

Conversation

@philipaconrad

Copy link
Copy Markdown
Member

What code changed, and why?

This PR adds a "rationale" file to our project to track design and implementation decisions over time. This can be helpful for understanding historical context around an API design, and may hep future contributors identify places where an earlier decision's assumptions no longer hold.

The file currently explains our external dependencies strategy, as well as the background behind the tools/SyncGen macro.

Related Resources

This commit adds a "rationale" file to our project to track design and
implementation decisions over time. This can be helpful for understanding
historical context around an API design, and may hep future contributors
identify places where an earlier decision's assumptions no longer hold.

The file currently explains our external dependencies strategy, as well as
the background behind the `tools/SyncGen` macro.

Signed-off-by: Philip Conrad <philip_conrad@apple.com>
@philipaconrad philipaconrad self-assigned this Sep 22, 2026
@philipaconrad philipaconrad added the docs Improvements or additions to documentation label Sep 22, 2026

@koponen koponen 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.

Looks good to me!

@philipaconrad
philipaconrad merged commit 911fb93 into open-policy-agent:main Sep 22, 2026
14 checks passed
Comment thread RATIONALE.md
For an infrequently-used development tool, we feel this is a reasonable hassle for *us* to deal with, but unreasonable for downstream users to have to grapple with.


## API Design

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.

Looks like the printer ate the rest of it 😉

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

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants