Skip to content

fix(ci): write step outputs to GITHUB_OUTPUT, set-output is deprecated - #22

Draft
openvoiceos-bot wants to merge 1 commit into
devfrom
fix/set-output-to-github-output
Draft

openvoiceos-bot wants to merge 1 commit into
devfrom
fix/set-output-to-github-output

Conversation

@openvoiceos-bot

Copy link
Copy Markdown

🤖 Auto-generated by Claude Opus 5 (claude-opus-5) via Claude Code — NOT human-reviewed. Verify before acting.

actionlint reports 7 findings on dev, and all 7 are the same one:

workflow command "set-output" was deprecated.
use `echo "{name}={value}" >> $GITHUB_OUTPUT` instead   [deprecated-commands]

GitHub disabled ::set-output, so the version step publishes nothing and every consumer of steps.version.outputs.version reads an empty string. In publish_alpha.yml that is the release tag and the release name.

Each line keeps its output name and its value byte for byte:

-        run: echo "::set-output name=version::$(python -c "from quebra_frases.version import __version__; print(__version__)")"
+        run: echo "version=$(python -c "from quebra_frases.version import __version__; print(__version__)")" >> $GITHUB_OUTPUT

No consumer moves, because the name version does not change.

Control

findings
actionlint on dev (e9218fb) 7, all set-output
actionlint after this change 0, exit 0

The rewritten line was also executed on its own, with a stub package, to prove the nested quoting survives: GITHUB_OUTPUT receives version=1.2.3a4.

This is the canary for the fleet sweep in T-5607: 44 repositories hold 111 of these lines. No workflow was dispatched — these are release workflows, and dispatching them publishes.

🤖 Generated with Claude Code

actionlint reports 7 findings on dev, all of them

    workflow command "set-output" was deprecated.
    use echo "{name}={value}" >> $GITHUB_OUTPUT instead

GitHub disabled the command, so the version step publishes nothing and every
consumer of steps.version.outputs.version reads an empty string.

The output name and the value are unchanged, so no consumer moves. actionlint
after the edit: 0 findings, exit 0. T-5607.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 28, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Scanning complete. No anomalies detected in the process. 🌌

I've aggregated the results of the automated checks for this PR below.

🔨 Build Tests

Verifying that everything is in its right place. 🧘

✅ All versions pass

Python Build Install Tests pytest
3.10 ✅ ✅ ✅ 22 passed in 0.10s
3.11 ✅ ✅ ✅ 22 passed in 0.09s
3.12 ✅ ✅ ✅ 22 passed in 0.05s
3.13 ✅ ✅ ✅ 22 passed in 0.10s
3.14 ✅ ✅ ✅ 22 passed in 0.11s

Making code review just a little bit easier. 💆‍♂️

This was referenced Sep 28, 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