Skip to content

Commit

Permalink
Refine templates (#155)
Browse files Browse the repository at this point in the history
- Use "<!-- ->" instead of "{...}" for place holders
- Add proposed text to adr-template-bare
- Re-add quotes around chosen option
  • Loading branch information
koppor authored Sep 5, 2024
1 parent d6ba5a9 commit d376c6f
Show file tree
Hide file tree
Showing 6 changed files with 19 additions and 15 deletions.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/).

## [Unreleased]

### Fixed

- `adr-template.md`: Re-add quotes around chosen option name.
- `adr-template-minimal.md`: Place holder in one line.
- `adr-template-bare.md`: Keep fixed template text as normal markdown (and not as comment).
- `adr-template-bare.md` and `adr-template-bare-minimal.md`: Always use `<!-- … ->` als place holder instead of `{…}`.

## [4.0.0-beta] – 2024-09-02

### Added
Expand Down
2 changes: 1 addition & 1 deletion docs/decisions/adr-template.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ title: ADR Template

## Decision Outcome

Chosen option: {title of option 1}, because {justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.
Chosen option: "{title of option 1}", because {justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.

<!-- This is an optional element. Feel free to remove. -->
### Consequences
Expand Down
4 changes: 1 addition & 3 deletions template/adr-template-bare-minimal.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@

# {short title, representative of solved problem and found solution}
# <!-- short title, representative of solved problem and found solution -->

## Context and Problem Statement

Expand All @@ -15,4 +14,3 @@

### Consequences


16 changes: 8 additions & 8 deletions template/adr-template-bare.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ consulted:
informed:
---

# {short title, representative of solved problem and found solution}
# <!-- short title, representative of solved problem and found solution -->

## Context and Problem Statement

Expand All @@ -22,23 +22,23 @@ informed:

## Decision Outcome


Chosen option: "", because

### Consequences

* <!-- * Good, because -->
* <!-- * Bad, because -->
* Good, because
* Bad, because

### Confirmation



## Pros and Cons of the Options

### {title of option}
### <!-- title of option -->

* <!-- * Good, because -->
* <!-- * Neutral, because -->
* <!-- * Bad, because -->
* Good, because
* Neutral, because
* Bad, because

## More Information
3 changes: 1 addition & 2 deletions template/adr-template-minimal.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@

## Decision Outcome

Chosen option: "{title of option 1}", because
{justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.
Chosen option: "{title of option 1}", because {justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.

<!-- This is an optional element. Feel free to remove. -->
### Consequences
Expand Down
2 changes: 1 addition & 1 deletion template/adr-template.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ informed: {list everyone who is kept up-to-date on progress; and with whom there

## Decision Outcome

Chosen option: {title of option 1}, because {justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.
Chosen option: "{title of option 1}", because {justification. e.g., only option, which meets k.o. criterion decision driver | which resolves force {force} | … | comes out best (see below)}.

<!-- This is an optional element. Feel free to remove. -->
### Consequences
Expand Down

0 comments on commit d376c6f

Please sign in to comment.