Skip to content

Document the release process - #212

Open
sensei-hacker wants to merge 3 commits into
iNavFlight:masterfrom
sensei-hacker:docs/release-process
Open

Document the release process#212
sensei-hacker wants to merge 3 commits into
iNavFlight:masterfrom
sensei-hacker:docs/release-process

Conversation

@sensei-hacker

Copy link
Copy Markdown
Member

Summary

Adds RELEASING.md, a maintainer-facing doc for the build/tag/publish process, and links it from CONTRIBUTING.md. Nothing in the existing docs (docs/*.md, which is the public mkdocs user site, or CONTRIBUTING.md) covered this before.

Written up after diagnosing why the 2.4 release shipped with zero assets (see #211): the tag didn't match the release.yml trigger, the workflow never built the _lua.zip asset in the first place, and the version string wasn't bumped before tagging. RELEASING.md documents the corrected process plus a "Lessons from the 2.4 release" section so the same failure mode isn't rediscovered from scratch next time.

Changes

  • RELEASING.md (new): versioning (single source of truth in iNav.lua), what make dist vs make dist-lua each do and why both are needed, how ci.yml/release.yml trigger, step-by-step release procedure, troubleshooting (moving a tag via the API, the restricted-GITHUB_TOKEN push block).
  • CONTRIBUTING.md: one-line pointer to RELEASING.md.

Test plan

  • Docs-only change; no build/CI impact expected.

Adds RELEASING.md covering versioning (single source of truth in
iNav.lua), what make dist / make dist-lua each build, how the CI and
release workflows trigger, the release procedure, and troubleshooting
for the two failure modes hit while shipping 2.4: tag not matching
the workflow trigger, and tagging before the version bump landed.
@pfeerick

Copy link
Copy Markdown

You also might want to add mention that the two packages need to be documented in the release notes also... i.e. this was missing in the release notes

Two Zip files are provided:

LuaTelemetry_v2.3.0.zip : Compiled LUA, should work on all radios (prior to EdgeTX 2.11rc1, which unfortunately breaks pre-compiled Lua).
LuaTelemetry v2.3.0_lua.zip : Source LUA, compiles on the device, may be required in the simulator. Also for EdgeTX 2.11rc1 and later.

Added section for Release Notes to guide users on changes and compatibility.
@sensei-hacker

Copy link
Copy Markdown
Member Author

You also might want to add mention that the two packages need to be documented in the release notes also... i.e. this was missing in the release notes

Thanks. Do you think this is good?:

https://github.com/sensei-hacker/OpenTX-Telemetry-Widget/blob/ea962f273dc0af1285121a43dd2277643cab408d/RELEASING.md#release-notes

Corrected grammatical errors in the release notes section.
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