fix: evaluate output of release-please correctly

[mandarini]

Summary

Fixes the release workflow condition that was causing the publish job to run even when no release was created.

Problem

The original workflow had this configuration:

outputs:
  should_publish: steps.release.outputs.release_created

# Later...
if: ${{ needs.release-please.outputs.should_publish }}

release_created returns the string "false" (not a boolean). When evaluated in the if condition with ${{ }}, GitHub Actions treats it as a JavaScript expression where any non-empty string is truthy - including the string "false". This caused the publish job to run even when no release was created.

Solution

Updated the workflow to explicitly compare string values:

  outputs:
    should_publish: ${{ steps.release.outputs.release_created == 'true' }}

  # Later...
  if: ${{ needs.release-please.outputs.should_publish == 'true' }}

Now the condition correctly evaluates:

• When no release: "false" == "true" → false → publish job skips
• When release created: "true" == "true" → true → publish job runs

Testing

See this repo: https://github.com/mandarini/test-release

Verified that:

• Publish job skips when pushing regular commits to main (only Release PR is created)
• Publish job runs when merging a Release PR (actual release is created)

[coveralls]

Pull Request Test Coverage Report for Build 18404061286

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 93.928%

---

Totals
Change from base Build 18386064843:	0.0%
Covered Lines:	8786
Relevant Lines:	9354

---

💛 - Coveralls