Skip to content

output buildId #89

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
andyshinn wants to merge 1 commit into
base: main
Choose a base branch
from
Open

output buildId #89

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
.idea
build_output.log
2 changes: 2 additions & 0 deletions action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,8 @@ inputs:
outputs:
manifest:
description: "The path to the generated manifest.vdf"
buildId:
description: "The Steam BuildID of the uploaded build"
Comment on lines +95 to +96
Copy link

Copilot AI Dec 7, 2025

Choose a reason for hiding this comment

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

The new buildId output is not documented in the README. Users should be informed about this new output and how to use it.

Consider adding a section in the README that documents the outputs available from this action, including both manifest and buildId. For example:

## Outputs

#### buildId

The Steam BuildID of the uploaded build. This can be used to reference the specific build in subsequent steps or for tracking purposes.

#### manifest

The path to the generated manifest.vdf file.

Copilot uses AI. Check for mistakes.
runs:
using: "docker"
image: Dockerfile
Expand Down
5 changes: 4 additions & 1 deletion steam_deploy.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ echo "# Uploading build #"
echo "#################################"
echo ""

steamcmd +login "$steam_username" +run_app_build "$manifest_path" +quit || (
steamcmd +login "$steam_username" +run_app_build "$manifest_path" +quit | tee build_output.log || (
echo ""
echo "#################################"
echo "# Errors #"
Expand Down Expand Up @@ -212,4 +212,7 @@ steamcmd +login "$steam_username" +run_app_build "$manifest_path" +quit || (
exit 1
)

buildId=$(grep -oP '\(BuildID \K\d+(?=\))' build_output.log || echo "")

echo "manifest=${manifest_path}" >> $GITHUB_OUTPUT
echo "buildId=${buildId}" >> $GITHUB_OUTPUT
Comment on lines +215 to +218
Copy link

Copilot AI Dec 7, 2025

Choose a reason for hiding this comment

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

The new buildId output lacks test coverage. The existing workflow in .github/workflows/main.yml should be updated to verify that the buildId output is correctly captured and set.

Consider adding a step after the steam-deploy action to check the output:

- id: deploy
  uses: ./
  with:
    # existing configuration
- run: |
    echo "Build ID: ${{ steps.deploy.outputs.buildId }}"
    if [ -z "${{ steps.deploy.outputs.buildId }}" ]; then
      echo "Warning: buildId output is empty"
    fi

Copilot uses AI. Check for mistakes.