Skip to content

Simplify resolve-version CLI by moving LTS defaults into binary#1620

Merged
colincasey merged 4 commits intomainfrom
simplified-resolve-version
Apr 21, 2026
Merged

Simplify resolve-version CLI by moving LTS defaults into binary#1620
colincasey merged 4 commits intomainfrom
simplified-resolve-version

Conversation

@colincasey
Copy link
Copy Markdown
Contributor

Summary

  • Remove lts_major_version CLI argument — the binary derives LTS major by resolving RECOMMENDED_LTS_VERSION against the inventory
  • Accept empty node_version to mean "use default LTS" — no more shell-side defaulting
  • Add default and lts_version fields to JSON output for shell script logging/warnings
  • Remove hardcoded lts_major_version="24" from lib/binaries.sh
  • Simplify fail_bin_install to take one argument

The binary now owns all version knowledge. The shell is a thin JSON consumer.

Replaces #1613 and #1616. Stacked on #1619.

@colincasey colincasey force-pushed the simplified-resolve-version branch from 4e1fba9 to 84f63c6 Compare April 21, 2026 12:40
@colincasey colincasey force-pushed the resolve-version-resolution-struct branch 2 times, most recently from 26b9163 to fcb7175 Compare April 21, 2026 13:05
@colincasey colincasey force-pushed the simplified-resolve-version branch from 84f63c6 to 2b9a2df Compare April 21, 2026 13:05
Base automatically changed from resolve-version-resolution-struct to main April 21, 2026 13:17
Remove hardcoded lts_major_version from shell scripts. The binary now
handles version defaulting and LTS derivation internally. Shell reads
'default' and 'lts_version' from JSON output for logging and warnings.
…nternally

The binary now accepts an empty node_version to mean 'use default LTS'.
LTS major is derived by resolving RECOMMENDED_LTS_VERSION against the
inventory. JSON output includes new 'default' and 'lts_version' fields.
@colincasey colincasey force-pushed the simplified-resolve-version branch from 2b9a2df to da584cf Compare April 21, 2026 13:23
@colincasey colincasey enabled auto-merge (squash) April 21, 2026 13:23
@colincasey colincasey merged commit 73ff5e4 into main Apr 21, 2026
22 checks passed
@colincasey colincasey deleted the simplified-resolve-version branch April 21, 2026 13:33
@heroku-linguist heroku-linguist Bot mentioned this pull request Apr 27, 2026
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