Skip to content

docs: add the release checklist, the notes composer and a changelog - #18

Merged
robrigo merged 1 commit into
mainfrom
docs/release-checklist
Aug 17, 2026
Merged

docs: add the release checklist, the notes composer and a changelog#18
robrigo merged 1 commit into
mainfrom
docs/release-checklist

Conversation

@robrigo

@robrigo robrigo commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Why

Releases of this contract were cut by hand and each body took its own shape, so a deployer hunted for the checksums, the ABI compatibility statement and the deploy notes in a different place every time. RELEASING.md fixes the procedure: the CHANGELOG entry rides in the feature PR, the release build writes SHA256SUMS with bare asset names and its two rows land in the entry as the release commit, a preview runs before the tag, and the Release attaches atomicmarket.wasm, atomicmarket.abi and SHA256SUMS. scripts/release-notes.sh composes the body from the entry at the tag plus the commit list since the previous release of the same kind and the compare link. CHANGELOG.md starts at 2.0.0 with the entry that matches the v2.0.0 Release now live, and attribution moves to AUTHORS.md. The existing Releases are already rewritten to this shape and the candidates retitled to their tags.

Validation

The two scripts are byte-identical to the reviewed atomicassets-api files; bash scripts/release-notes.test.sh passes 12/12 and CI runs it; shellcheck is clean. git rev-list --count confirms the v2.0.0 range the live body lists, and a preview from a scratch branch carrying a draft entry composed the body as the checklist describes.

@robrigo
robrigo requested a lite review from Copilot August 17, 2026 20:55

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI 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.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

Suppressed comments (2)

AUTHORS.md:12

  • Avoid splitting the emphasized author handle across lines; keeping the bold span contiguous makes the rendered output more predictable.
Most of the v2 contract was authored by **t-break
([@on-a-t-break](https://github.com/on-a-t-break))**, and reached this

AUTHORS.md:7

  • Avoid splitting bold text across a newline; it can render inconsistently depending on Markdown renderer. Consider keeping the emphasized name together and wrapping outside the ... span.

This issue also appears on line 11 of the same file.

current owner and maintainer. The v1 contract was written by **Jona Wilmsmann
(Pink Network)**.

Releases were cut by hand and each Release body was written in its own shape,
so a deployer had to hunt for the checksums, the ABI compatibility statement and
the deploy notes in a different place every time, and attribution repeated per
release. RELEASING.md fixes the procedure: the CHANGELOG entry rides in the
feature PR, the release build writes SHA256SUMS with bare asset names and its
two rows land in the entry as the release commit, a preview runs before the tag,
and the Release attaches the wasm, the abi and SHA256SUMS. scripts/release-notes.sh
(the composer shared with atomicassets-api, copied unchanged with its paired
test that CI now runs) composes the body from the entry at the tag plus the
commit list since the previous release of the same kind and the compare link.
CHANGELOG.md starts at 2.0.0 with the entry that matches the live v2.0.0
Release; the README points at both files and at AUTHORS.md, which carries the
lineage the Release bodies used to repeat.
@robrigo

robrigo commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Round 2 suppressed notes on AUTHORS.md: applied, the two bold spans no longer wrap across a line.

@robrigo
robrigo force-pushed the docs/release-checklist branch from f263d1f to c40c777 Compare August 17, 2026 21:04
@robrigo
robrigo merged commit a1a279c into main Aug 17, 2026
1 check passed
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.

2 participants