Skip to content

[PM-27192] Fix version-info.json values - #2056

Merged
vvolkgang merged 1 commit into
mainfrom
vvolkgang/fix-version-info
Oct 21, 2025
Merged

[PM-27192] Fix version-info.json values#2056
vvolkgang merged 1 commit into
mainfrom
vvolkgang/fix-version-info

Conversation

@vvolkgang

@vvolkgang vvolkgang commented Oct 21, 2025

Copy link
Copy Markdown
Member

🎟️ Tracking

PM-27192

📔 Objective

After #2014 version-info.json had the var names instead of the version values.

📸 Screenshots

Before After
image image

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

@vvolkgang
vvolkgang requested a review from a team as a code owner October 21, 2025 15:35
@vvolkgang
vvolkgang enabled auto-merge (squash) October 21, 2025 15:36
@github-actions

Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailscef7b3c9-b318-4561-817b-28022cc4abd5

Great job! No new security vulnerabilities introduced in this pull request

@codecov

codecov Bot commented Oct 21, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.97%. Comparing base (d6cd64f) to head (7ace6e8).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2056      +/-   ##
==========================================
- Coverage   85.31%   82.97%   -2.34%     
==========================================
  Files        1691     1988     +297     
  Lines      144372   161209   +16837     
==========================================
+ Hits       123168   133760   +10592     
- Misses      21204    27449    +6245     

☔ View full report in Codecov by Sentry.
📢 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.

@vvolkgang
vvolkgang merged commit 961eeaf into main Oct 21, 2025
15 of 17 checks passed
@vvolkgang
vvolkgang deleted the vvolkgang/fix-version-info branch October 21, 2025 17:03
@vvolkgang vvolkgang added the t:ci Change Type - Updates to automated workflows label Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:ci Change Type - Updates to automated workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants