[PR #1350] [CLOSED] feat: DRYRUN job output comments #1895

New issue

Closed

opened 2026-03-01 21:53:08 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 21:53:08 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/1350
Author: @ChristopherHX
Created: 9/15/2022
Status: ❌ Closed

Base: master ← Head: feat-dryrun-job-outputs

📝 Commits (5)

92642bf feat: DRYRUN job output comments
5e1b291 fix lint
aa9ab6b fix name collision
063e55a Update workflow.go
7a727d0 fix: crash + tests

📊 Changes

4 files changed (+70 additions, -0 deletions)

View changed files

📝 pkg/model/planner_test.go (+1 -0)
➕ pkg/model/testdata/dryrunoutputcomment/push.yml (+27 -0)
📝 pkg/model/workflow.go (+38 -0)
📝 pkg/runner/run_context.go (+4 -0)

📄 Description

Ideas are welcome, this is already working

Run with --dryrun, it throws before this change

on: push
jobs:
  list-manifests:
    runs-on: ubuntu-latest
    outputs:
      # Comment for this output
      # DRYRUN: ["7.3", "7.4"]
      # Another Comment for this output
      matrix: ${{ steps.set-matrix.outputs.matrix }}
    steps:
    - run: echo Hi
  otherjob:
    needs: list-manifests
    runs-on: ubuntu-latest
    strategy:
      matrix:
        X: ${{ fromjson(needs.list-manifests.outputs.matrix) }}
    steps:
    - run: echo Hi

Discuss this feature
Add Test

Fixes #1347

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/nektos/act/pull/1350 **Author:** [@ChristopherHX](https://github.com/ChristopherHX) **Created:** 9/15/2022 **Status:** ❌ Closed **Base:** `master` ← **Head:** `feat-dryrun-job-outputs` --- ### 📝 Commits (5) - [`92642bf`](https://github.com/nektos/act/commit/92642bf77b9e735713cc4534af24ca7bf20a739e) feat: DRYRUN job output comments - [`5e1b291`](https://github.com/nektos/act/commit/5e1b291e02f254e45551dc091a4a31ccf0e61300) fix lint - [`aa9ab6b`](https://github.com/nektos/act/commit/aa9ab6bbc2b3c4df74bc973b981d2d3c356927e8) fix name collision - [`063e55a`](https://github.com/nektos/act/commit/063e55abb8e5c0103476e4f46b25d6ab9fcc5866) Update workflow.go - [`7a727d0`](https://github.com/nektos/act/commit/7a727d099756f8e3b713d19dbaaa8a76c733e5aa) fix: crash + tests ### 📊 Changes **4 files changed** (+70 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `pkg/model/planner_test.go` (+1 -0) ➕ `pkg/model/testdata/dryrunoutputcomment/push.yml` (+27 -0) 📝 `pkg/model/workflow.go` (+38 -0) 📝 `pkg/runner/run_context.go` (+4 -0) </details> ### 📄 Description Ideas are welcome, this is already working Run with `--dryrun`, it throws before this change ``` on: push jobs: list-manifests: runs-on: ubuntu-latest outputs: # Comment for this output # DRYRUN: ["7.3", "7.4"] # Another Comment for this output matrix: ${{ steps.set-matrix.outputs.matrix }} steps: - run: echo Hi otherjob: needs: list-manifests runs-on: ubuntu-latest strategy: matrix: X: ${{ fromjson(needs.list-manifests.outputs.matrix) }} steps: - run: echo Hi ``` - [ ] Discuss this feature - [x] Add Test Fixes #1347 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-01 21:53:08 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-03-01 21:54:20 +03:00

[PR #1895] [MERGED] Remove archives.replacements in goreleaser.yaml #2179

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/act#1895

No description provided.

Rows
Columns