Skip to content

Comments

Update master branch build to release timestamp version to the github packages#25

Merged
ayeshLK merged 3 commits intoballerina-platform:mainfrom
ayeshLK:main
Feb 13, 2026
Merged

Update master branch build to release timestamp version to the github packages#25
ayeshLK merged 3 commits intoballerina-platform:mainfrom
ayeshLK:main

Conversation

@ayeshLK
Copy link
Member

@ayeshLK ayeshLK commented Feb 13, 2026

Purpose

$subject

Summary

This pull request updates the master branch build workflow to use a new template specifically designed for releasing timestamped versions to GitHub Packages.

Changes

Workflow Configuration Update

  • Updated .github/workflows/build-timestamped-master.yml to reference the build-timestamp-master-template.yml template instead of the generic build-connector-template.yml template
  • This change enables the master branch builds to produce and publish releases with timestamp-based versioning to GitHub Packages
  • The workflow inputs and overall control flow remain consistent with existing implementations

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

A GitHub Actions workflow file was updated to reference a different template workflow. The call_workflow job now invokes build-timestamp-master-template.yml instead of build-connector-template.yml, with all inputs and parameters preserved.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/build-timestamped-master.yml
Updated workflow template reference from build-connector-template.yml to build-timestamp-master-template.yml in the call_workflow job invocation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A template swap so swift and clean,
From connector to timestamp—new scene!
One line in, one line out with care,
The workflow dances through the air!

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and fails to follow the required template structure. It contains only a placeholder ("> $subject") instead of a meaningful Purpose section, and completely omits other required sections like Examples and the Checklist. Provide a complete PR description following the template: include a meaningful Purpose section explaining the changes, relevant Examples if applicable, and complete the Checklist with appropriate selections.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (1 file):

⚔️ .github/workflows/build-timestamped-master.yml (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating the master branch build workflow to release a timestamp version to GitHub packages, which matches the workflow template update from build-connector-template.yml to build-timestamp-master-template.yml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch main
  • Post resolved changes as copyable diffs in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.github/workflows/build-timestamped-master.yml (1)

14-18: ⚠️ Potential issue | 🔴 Critical

Remove the unsupported repo-name input from the workflow call.

The template build-timestamp-master-template.yml only accepts additional-build-flags as an input parameter. The repo-name input being passed is not defined in the template and will cause the workflow to fail. Remove repo-name: module-ballerinax-cdc from the with section.

Current configuration:
    uses: ballerina-platform/ballerina-library/.github/workflows/build-timestamp-master-template.yml@main
    secrets: inherit
    with:
      repo-name: module-ballerinax-cdc
      additional-build-flags: -x :cdc-examples:build

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.70%. Comparing base (23ca6d3) to head (e305e94).
⚠️ Report is 5 commits behind head on main.

❌ Your project status has failed because the head coverage (76.70%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #25   +/-   ##
=========================================
  Coverage     76.70%   76.70%           
  Complexity      224      224           
=========================================
  Files            30       30           
  Lines          1082     1082           
  Branches        169      169           
=========================================
  Hits            830      830           
  Misses          175      175           
  Partials         77       77           

☔ 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.

@ayeshLK ayeshLK merged commit 4b25234 into ballerina-platform:main Feb 13, 2026
5 of 6 checks passed
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.

2 participants