Skip to content

Conversation

@vvolkgang
Copy link
Member

🎟️ Tracking

PM-28041

📔 Objective

The script used to create SDK Update PRs limited the changelog list to the last 20 PRs. We're removing this limitation now.

⏰ 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 added the t:ci Change Type - Updates to automated workflows label Nov 10, 2025
@claude
Copy link
Contributor

claude bot commented Nov 10, 2025

Claude finished @vvolkgang's task —— View job


Overall Assessment: APPROVE

This is a straightforward script change that successfully removes the 20-commit limit from the changelog generation. The change achieves its stated objective cleanly.

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.99%. Comparing base (4032d2b) to head (bb0e0ed).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6152   +/-   ##
=======================================
  Coverage   84.99%   84.99%           
=======================================
  Files         723      723           
  Lines       52740    52740           
  Branches     7649     7649           
=======================================
  Hits        44824    44824           
  Misses       5243     5243           
  Partials     2673     2673           

☔ 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 enabled auto-merge November 10, 2025 21:36
@github-actions
Copy link
Contributor

Logo
Checkmarx One – Scan Summary & Detailsa246d541-6bfa-45f0-a78b-441108c80f29

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

@vvolkgang vvolkgang added this pull request to the merge queue Nov 10, 2025
Merged via the queue into main with commit 7a6a493 Nov 10, 2025
13 checks passed
@vvolkgang vvolkgang deleted the vvolkgang/changelog branch November 10, 2025 22:00
dev-sharma3624 pushed a commit to dev-sharma3624/android that referenced this pull request Nov 12, 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.

3 participants