Skip to content

Fix tera installation and usage - #1238

Merged
ladvoc merged 1 commit into
mainfrom
ladvoc/fix-tera
Jul 10, 2026
Merged

Fix tera installation and usage#1238
ladvoc merged 1 commit into
mainfrom
ladvoc/fix-tera

Conversation

@ladvoc

@ladvoc ladvoc commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Issue: chevdor/tera-cli and teracli (from crates.io) are not the same, and the latter is unmaintained. This PR uses the maintained version and updates CLI args accordingly.

Remove existing installation from your local system as follows to allow the new one to be installed:

rm $(which tera)

@ladvoc
ladvoc requested a review from pblazej as a code owner July 9, 2026 22:16
@ladvoc ladvoc added the internal to tag changes that don't require changelog documentation label Jul 9, 2026
@ladvoc
ladvoc force-pushed the ladvoc/fix-tera branch from 39f7717 to cbaaa2c Compare July 9, 2026 22:18

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@ladvoc
ladvoc requested a review from jhugman July 9, 2026 22:21
@ladvoc
ladvoc merged commit ce46ae0 into main Jul 10, 2026
27 of 28 checks passed
@ladvoc
ladvoc deleted the ladvoc/fix-tera branch July 10, 2026 14:12
pblazej added a commit that referenced this pull request Jul 30, 2026
The composite action pre-installed crates.io `tera-cli@0.5.0`, which is a
different project from chevdor/tera-cli and has a different CLI (`--env`,
no `--env-only`). Because it still provides a `tera` binary, cargo-make's
`tera --help` probe succeeded and the `install-tera` task never ran, so
swift-generate-manifest died on "Found argument '--env-only' which wasn't
expected" — reintroducing the bug #1238 had already fixed.

There is no binary distribution to pre-install: chevdor publishes as
`teracli`, and only up to 0.3.0. Drop the step and let `install-tera` build
from the pinned git tag (~30s).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal to tag changes that don't require changelog documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants