Skip to content

Bump version to 5.4.2 - #59

Merged
dorin-ga merged 2 commits into
mainfrom
release/v5.4.2
Nov 3, 2026
Merged

dorin-ga merged 2 commits into
mainfrom
release/v5.4.2

Conversation

@dorin-ga

@dorin-ga dorin-ga commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Version bump ahead of the v5.4.2 release.

  • GA_VERSION_STR → cpp 5.4.2 (source/gameanalytics/GACommon.h)
  • CHANGELOG.md: ## 5.4.2
  • release.py: new release script (prepare / publish)
  • .github/workflows/create_release.yml: the tag step reuses an existing tag if it points at the commit being built

Fixed

  • Linux: build used -fPIC
  • Windows: cpu model detection has been improved
  • Windows: windows version is now more reliable

Once merged, tag and release with python release.py publish 5.4.2.

@dorin-ga dorin-ga added the ignore-for-release Ignore this PR from release notes label Nov 3, 2026
release.py prepare bumps GA_VERSION_STR, updates CHANGELOG.md and opens
the release PR; release.py publish tags the merged PR and triggers the
Create Release workflow via gh_create_release.py.

The workflow's tag step now reuses an existing tag when it points at
the commit being built, since release.py pushes the tag first.
@dorin-ga
dorin-ga requested a review from Gogoshika-ga November 3, 2026 10:55
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.79%. Comparing base (0c1c055) to head (28c016f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   58.36%   58.79%   +0.42%     
==========================================
  Files          23       23              
  Lines        3060     3060              
  Branches     1512     1512              
==========================================
+ Hits         1786     1799      +13     
+ Misses        951      939      -12     
+ Partials      323      322       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Dispatch errors are printed without a nonzero exit, so publishing can appear successful after pushing the release tag even though GitHub rejected the release build dispatch.

1 open issue
What changed in this PR

Prepares the C++ SDK for version 5.4.2 and adds tooling to prepare release PRs and publish merged releases.

Changes:

  • Updates the SDK version and records Linux and Windows fixes in the changelog.
  • Adds release.py with prepare and publish commands.
  • Allows the release workflow to reuse a tag when it points to the commit being built.
File Description
source/​gameanalytics/​GACommon.h Updates the SDK version to cpp 5.4.2.
release.py Adds release preparation, tagging, and workflow dispatch.
CHANGELOG.md Adds the 5.4.2 release notes.
.github/​workflows/​create_release.yml Validates and reuses existing release tags.

Review effort: Balanced

Comment thread release.py
if run("git", "rev-parse", f"origin/{BASE_BRANCH}") != commit:
sys.exit(f"origin/{BASE_BRANCH} has moved past {tag}, so the release build would not match the tag. "
f"Workflow not triggered.")
trigger_workflow(tag)
@dorin-ga
dorin-ga merged commit 82b7a08 into main Nov 3, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release Ignore this PR from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants