Skip to content

ADR-0001 Bootstrap Architecture Decision Records#4

Merged
jhagberg merged 5 commits into
mainfrom
adr-init
Dec 10, 2025
Merged

ADR-0001 Bootstrap Architecture Decision Records#4
jhagberg merged 5 commits into
mainfrom
adr-init

Conversation

@jhagberg

Copy link
Copy Markdown
Contributor

Bootstrap Architecture Decision Records

Initialize the repository with adr-tools and document our decision to use ADRs.

Changes

  • Initialize adr-tools pointing to docs/decisions/
  • Create ADR-0001: Record architecture decisions

References

@jhagberg jhagberg requested a review from a team November 25, 2025 14:45
@jhagberg jhagberg changed the title Bootstrap Architecture Decision Records ADR-00001 Bootstrap Architecture Decision Records Nov 25, 2025
@jhagberg jhagberg changed the title ADR-00001 Bootstrap Architecture Decision Records ADR-0001 Bootstrap Architecture Decision Records Nov 25, 2025
@jhagberg

Copy link
Copy Markdown
Contributor Author

Or should we use the MADR instead https://adr.github.io/adr-templates/

@jwindhager jwindhager left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Or should we use the MADR instead https://adr.github.io/adr-templates/

What's the difference?

Comment thread .gitignore Outdated
@jhagberg

jhagberg commented Dec 2, 2025

Copy link
Copy Markdown
Contributor Author

LGTM

Or should we use the MADR instead https://adr.github.io/adr-templates/

What's the difference?

There is a full https://github.com/adr/madr/blob/4.0.0/template/adr-template.md?plain=1 and a minimal version https://github.com/adr/madr/blob/4.0.0/template/adr-template-minimal.md?plain=1

a bit more information but we can keep it minimal using the suggested template.

@Netzach Netzach self-requested a review December 8, 2025 09:45

@Netzach Netzach left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good

@alneberg alneberg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine with me.

@jwindhager jwindhager left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM (requested changes are minor & optional)

Comment thread docs/decisions/0001-record-architecture-decisions.md Outdated
Comment thread README.md Outdated
…nd script for formatting identifiers according to review comment
@jhagberg jhagberg requested a review from jwindhager December 9, 2025 11:07
@jhagberg jhagberg requested a review from jwindhager December 9, 2025 11:26
@jhagberg jhagberg merged commit a210ee4 into main Dec 10, 2025
1 check passed
@jhagberg jhagberg deleted the adr-init branch December 10, 2025 09:41
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.

5 participants