Skip to content

[TE-10262]: version lt-reports fatjar publish & drop CI on push#18

Merged
saurabh-prakash merged 2 commits into
lt-reportsfrom
TE-10262-lt-reports-versioning
Jun 26, 2026
Merged

[TE-10262]: version lt-reports fatjar publish & drop CI on push#18
saurabh-prakash merged 2 commits into
lt-reportsfrom
TE-10262-lt-reports-versioning

Conversation

@saurabh-prakash

Copy link
Copy Markdown
Collaborator

What

Two fixes for the lt-reports report-fatjar pipeline:

  1. Fix the publish 409. lt-reports-cd published under a fixed coordinate (com.lambdatest:karate-reports:lt-reports). GitHub Packages treats released versions as immutable, so every run after the first failed with HTTP 409 Conflict (example run). It now publishes under ${project.version}.${lt.patch.version} (e.g. 1.5.3.1).
  2. Drop CI on push. lt-reports-ci now triggers only on pull_request and workflow_dispatch (CD still publishes on push to lt-reports). Supersedes the closed Update CI workflow to remove push trigger #17.

Versioning scheme

  • karate-parent <version> keeps tracking upstream karate unchanged (1.5.3).
  • New lt.patch.version property in the root pom is a separate LambdaTest-fork counteradded, not appended to the karate version, so the Maven Central release coordinate stays clean.
  • Publish version = project.version + . + lt.patch.version.

To republish after a change: bump lt.patch.version in the root pom.

⚠️ Downstream note

The publish coordinate now changes per release, so hyperexecute-postprocessing-service (which pulled the fixed lt-reports URL) must be pointed at the bumped version after each publish. First publish from this branch lands at 1.5.3.1.

🤖 Generated with Claude Code

GitHub Packages rejects re-publishing an immutable version (HTTP 409),
so the fixed `lt-reports` coordinate failed on every CD run after the
first. Publish under ${project.version}.${lt.patch.version} instead:
project.version tracks upstream karate unchanged, lt.patch.version is a
new LambdaTest-fork counter in the root pom (added, not appended, so the
Maven Central release coordinate stays clean). Bump it to republish.

Also drop the push trigger on lt-reports-ci so the build check runs only
on PRs and manual dispatch (CD still publishes on push to lt-reports).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@saurabh-prakash saurabh-prakash force-pushed the TE-10262-lt-reports-versioning branch from a879619 to 8c90199 Compare June 26, 2026 10:30
@saurabh-prakash saurabh-prakash force-pushed the TE-10262-lt-reports-versioning branch from 8c90199 to a724e43 Compare June 26, 2026 10:33
@saurabh-prakash saurabh-prakash merged commit d656e31 into lt-reports Jun 26, 2026
1 check 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.

1 participant