8000 fix(ci): skip Docker/artifact uploads for pre-releases by ysfscream · Pull Request #1919 · emqx/MQTTX · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(ci): skip Docker/artifact uploads for pre-releases #1919

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 10, 2025
Merged

Conversation

ysfscream
Copy link
Member

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Currently, the CI/CD workflows for building packages, the CLI, and the web application trigger deployment/publishing actions (like uploading to AWS S3, publishing Docker images) for all GitHub releases, including pre-releases.

Issue Number

N/A

What is the new behavior?

The GitHub Actions workflows (build_packages.yaml, build_cli.yaml, deploy_web.yaml) have been updated. Now, steps related to deploying to external services (AWS S3, Docker Hub) will only run for full releases (release event where prerelease is false). Actions like publishing to npm and deploying the website were already correctly configured. Uploading artifacts directly to the GitHub Release page remains unchanged and will happen for both pre-releases and full releases.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

These changes modify the conditional logic within the GitHub Actions workflow files in the .github/workflows/ directory to differentiate between pre-releases and full releases for deployment steps. Reviewers should check the if conditions added to the relevant jobs/steps.

Other information

This change ensures that pre-releases do not trigger production deployments to AWS S3 or Docker Hub, allowing for safer testing and staging via GitHub pre-releases.

@ysfscream ysfscream added fix Fix bug or issues chore Changes in build tools or dependent packages CI/CD labels Apr 9, 2025
@ysfscream ysfscream added this to the v1.12.0 milestone Apr 9, 2025
@ysfscream ysfscream requested review from Red-Asuka and Copilot April 9, 2025 17:17
@ysfscream ysfscream added this to MQTTX Apr 9, 2025
Copy link
@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

@ysfscream ysfscream self-assigned this Apr 10, 2025
@ysfscream ysfscream moved this to In Progress in MQTTX Apr 10, 2025
@Red-Asuka Red-Asuka merged commit 867cb7c into main Apr 10, 2025
4 checks passed
@Red-Asuka Red-Asuka deleted the ysf/desktop branch April 10, 2025 03:23
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQTTX Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes in build tools or dependent packages CI/CD fix Fix bug or issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants
0