Skip to content

Commit 32ad2fd

Browse files
infracoreclaude
andcommitted
update: README with new PR comment format, spec mode, live demo link
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent d252a99 commit 32ad2fd

1 file changed

Lines changed: 29 additions & 19 deletions

File tree

README.md

Lines changed: 29 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -4,51 +4,61 @@ Fork this repo, push a breaking API change, and see Delimit catch it in CI.
44

55
## What's in the box
66

7-
- `openapi.yaml` -- a Pet Store API with 4 endpoints
8-
- `openapi-changed.yaml` -- the same spec with breaking changes:
7+
- `openapi.yaml` a Pet Store API with 4 endpoints
8+
- `openapi-changed.yaml` the same spec with 5 breaking changes:
99
- `GET /pets/{petId}` removed
1010
- `DELETE /pets/{petId}` removed
1111
- `Pet.id` type changed from `string` to `integer`
1212
- `Pet.tag` field removed
1313
- `Pet.status` enum value `pending` removed
14-
- `.github/workflows/delimit.yml` -- runs the Delimit action on every push/PR
14+
- `.github/workflows/delimit.yml` runs the Delimit action on every push/PR
1515

1616
## Try it
1717

1818
1. Fork this repo
19-
2. Push any commit (or just trigger Actions manually)
20-
3. Watch CI fail:
19+
2. Push any commit (or trigger Actions manually)
20+
3. On a PR, Delimit posts a comment like this:
2121

22-
```
23-
::error title=Delimit::Endpoint /pets/{petId} removed
24-
::error title=Delimit::Field id type changed from string to integer
25-
```
22+
> **Breaking API Changes Detected**
23+
>
24+
> | Severity | Change | Location |
25+
> |----------|--------|----------|
26+
> | Critical | Endpoint removed | `/pets/{petId}` |
27+
> | Warning | Type changed (string → integer) | `Pet.id` |
28+
> | Warning | Enum value removed | `Pet.status` |
29+
>
30+
> **Semver: MAJOR** — 5 breaking changes, migration guide included
2631
27-
4. On a PR, Delimit posts a comment with a migration guide
32+
4. In enforce mode, CI fails until the breaking changes are resolved
2833

29-
## Fix the build
34+
## Use Delimit in your own repo
3035

31-
Delete `openapi-changed.yaml` and update the workflow to compare `openapi.yaml` against itself. Or better -- edit `openapi-changed.yaml` to only add new endpoints and fields. Additive changes pass.
36+
The simplest setup — auto-detects the base branch and diffs your spec:
3237

33-
## Use Delimit in your own repo
38+
```yaml
39+
- uses: delimit-ai/delimit-action@v1
40+
with:
41+
spec: api/openapi.yaml
42+
```
43+
44+
Or compare two specific files:
3445
3546
```yaml
36-
# .github/workflows/delimit.yml
3747
- uses: delimit-ai/delimit-action@v1
3848
with:
3949
old_spec: openapi.yaml
40-
new_spec: openapi.yaml
50+
new_spec: openapi-changed.yaml
4151
mode: enforce
4252
```
4353
44-
Point `old_spec` at your base spec and `new_spec` at the changed version. The action detects breaking changes and posts PR comments automatically.
45-
46-
Or use the CLI:
54+
CLI:
4755
4856
```
4957
npx delimit-cli lint openapi.yaml
5058
```
5159

60+
No config. No API keys. No account.
61+
5262
---
5363

54-
[GitHub Action](https://github.com/delimit-ai/delimit-action) | [CLI](https://www.npmjs.com/package/delimit-cli) | [delimit.ai](https://delimit.ai)
64+
[Delimit](https://delimit.ai) | [GitHub Action](https://github.com/marketplace/actions/delimit-api-governance) | [CLI](https://www.npmjs.com/package/delimit-cli) | [Live Demo](https://github.com/delimit-ai/delimit-action-demo/pull/2)

0 commit comments

Comments
 (0)