Skip to content

patch#1946

Merged
0x4C756B65 merged 2 commits into
mainfrom
shankar/ci
Jun 30, 2026
Merged

patch#1946
0x4C756B65 merged 2 commits into
mainfrom
shankar/ci

Conversation

@shankars99

Copy link
Copy Markdown
Collaborator
  1. update tj-actions/changed-files

  2. do not test before publish

Signed-off-by: shankar <shankar@layerzerolabs.org>
Signed-off-by: shankar <shankar@layerzerolabs.org>
@cursor

cursor Bot commented Jun 30, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Publishing on main no longer gates on the reusable test workflow in this pipeline, which increases the chance of shipping without those checks unless PR or other workflows still cover them.

Overview
CI workflow adjustments for bot auto-approval and main-branch publishing.

The auto-approve workflow for layerzero-bot bumps tj-actions/changed-files from v44 to v46.0.1; allowed-file patterns and approval logic are unchanged.

On main pushes, the check job that called reusable-test.yaml is removed, and publish no longer needs: check, so NPM publish can run immediately on merge without a green test run in this workflow. The workflow is still named Check & publish; downstream update-lockfiles still triggers on Check & publish completion.

Reviewed by Cursor Bugbot for commit c47c628. Bugbot is set up for automated code reviews on this repo. Configure here.

@St0rmBr3w St0rmBr3w left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedgithub/​tj-actions/​changed-files@​2d756ea4c53f7f6b397767d8723b3a10a9f35bf2 ⏵ 2f7c5bfce28377bc069a65ba478de0a74aa0ca3299100100100100

View full report

@github-actions

Copy link
Copy Markdown
Contributor

🧪 E2E Test Status

E2E tests are non-blocking and validate real blockchain interactions. Failures may occur due to network issues, RPC rate limits, or external service downtime.

Test Runs (Newest First):

  • Run #6774 - Failed - 2026-06-30 17:32 (UTC)

@0x4C756B65 0x4C756B65 merged commit abc569c into main Jun 30, 2026
20 of 23 checks passed
@0x4C756B65 0x4C756B65 deleted the shankar/ci branch June 30, 2026 19:52
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.

4 participants