Skip to content

Fix the assembly stamping of the base build after moving to gha#478

Merged
jasonleenaylor merged 1 commit into
release/9.3from
fix/patching
Sep 29, 2025
Merged

Fix the assembly stamping of the base build after moving to gha#478
jasonleenaylor merged 1 commit into
release/9.3from
fix/patching

Conversation

@jasonleenaylor

@jasonleenaylor jasonleenaylor commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

This change is Reviewable

Copilot AI review requested due to automatic review settings September 29, 2025 17:36

Copilot AI 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.

Pull Request Overview

This PR updates the build system to work with GitHub Actions (GHA) by replacing Jenkins-specific environment variable references with GHA equivalents for assembly stamping.

  • Updates environment variable names from Jenkins conventions to GitHub Actions conventions
  • Changes BUILD_NUMBER to RELEASE_BASE_BUILD_NUMBER for build numbering
  • Replaces BaseBuildish with BASE_BUILD_NUMBER for patch build detection

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread Build/Src/FwBuildTasks/Substitute.cs Outdated

@KenZook KenZook 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.

@KenZook reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed (commit messages unreviewed), 1 unresolved discussion (waiting on @jasonleenaylor)

@jasonleenaylor jasonleenaylor merged commit 4bd20a8 into release/9.3 Sep 29, 2025
4 of 5 checks passed
@jasonleenaylor jasonleenaylor deleted the fix/patching branch September 29, 2025 18:36
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.

3 participants