Skip to content

Publish trigger#1104

Merged
hankliu418 merged 3 commits intohotfix-publish-issuefrom
publish_trigger
Feb 5, 2026
Merged

Publish trigger#1104
hankliu418 merged 3 commits intohotfix-publish-issuefrom
publish_trigger

Conversation

@hankliu418
Copy link
Contributor

@hankliu418 hankliu418 commented Feb 5, 2026


Checklist:

  • README.md includes:
    • The details of signing data and transaction data, this includes parameters.
    • Official docs or white paper.
    • Website or api can query assets and broadcast transaction.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

PR Summary by Typo

Overview

This PR addresses an issue in the publishing action by correcting the npm 404 error code string, which was preventing proper detection of unpublished packages. It also enhances debugging capabilities by logging the raw output of the npm view command.

Key Changes

  • Corrected the NPM_404_ERR_CODE constant to accurately match the npm ERR! code E404 message.
  • Added a console.log statement to display the raw output of the npm view command in the isLocalUpgraded function, aiding in debugging.

Work Breakdown

Category Lines Changed
New Work 4 (26.7%)
Churn 1 (6.7%)
Rework 10 (66.7%)
Total Changes 15
To turn off PR summary, please visit Notification settings.

@typo-app
Copy link

typo-app bot commented Feb 5, 2026

Static Code Review 📊

✅ All quality checks passed!

@hankliu418 hankliu418 merged commit 67937f0 into hotfix-publish-issue Feb 5, 2026
0 of 2 checks passed
@hankliu418 hankliu418 deleted the publish_trigger branch February 5, 2026 13:00
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 5, 2026

@cbx-tech cbx-tech requested review from a team, Rickuan, a0979470582, gundomlegend, jin860315, melody0000tw and tp26610 and removed request for a team February 5, 2026 13:02
Copy link

@typo-app typo-app bot left a comment

Choose a reason for hiding this comment

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

AI Code Review 🤖

Files Reviewed: 2
Comments Added: 0
Lines of Code Analyzed: 25
Critical Issues: 0

PR Health: Excellent 🔥

Give 👍 or 👎 on each review comment to help us improve.

@cbx-tech cbx-tech requested review from Dorac and removed request for a team February 5, 2026 13:02
hankliu418 added a commit that referenced this pull request Feb 6, 2026
* update publish action

* fix - publish issue (#1103)

* add console log

* update console content

---------

Co-authored-by: Hank Liu <hankliu@coolbitx.com>

* Publish trigger (#1104)

* add console log

* update console content

* update err code adjudgement

---------

Co-authored-by: Hank Liu <hankliu@coolbitx.com>

* fix(zec): fix zec build error (#1105)

* add console log

* update console content

* update err code adjudgement

* fix build issue

* remove debug log in publish action

---------

Co-authored-by: Hank Liu <hankliu@coolbitx.com>

* remove redundant import and log

---------

Co-authored-by: Hank Liu <hankliu@coolbitx.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant