+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

fix(ci): fix the publishing job in the CLI release workflow #3489

Merged
merged 1 commit into from
Oct 26, 2022
Merged

Conversation

leops
Copy link
Contributor

@leops leops commented Oct 26, 2022

Summary

This PR fixes the publish jobs of the release_cli workflow by:

  • Filtering out the @rometools/js-api package from the list of published packages, since the script automatically publishes all the packages under npm/ but js-api is released from its own workflow
  • Changed the release environment from marketplace to npm-publish, this was an historical artifact since packages used to be pushed to npm and the VS Marketplace from the same workflow but this is no longer the case

Test Plan

Run the workflow manually and publish a new nightly for the CLI

@leops leops requested a review from a team October 26, 2022 07:52
@netlify
Copy link

netlify bot commented Oct 26, 2022

Deploy Preview for rometools canceled.

Name Link
🔨 Latest commit 75134eb
🔍 Latest deploy log https://app.netlify.com/sites/rometools/deploys/6358e73b83d7e9000948e065

@leops leops merged commit e8dd9b4 into main Oct 26, 2022
@leops leops deleted the fix/ci-npm branch October 26, 2022 08:36
@leops leops added the A-Tooling Area: our own build, development, and release tooling label Oct 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Tooling Area: our own build, development, and release tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载