Skip to content

chore(ci): add fallback value for scheduled workflow run#11641

Merged
stormslowly merged 1 commit intomainfrom
chore/fix_crontab_canary_workflow
Sep 11, 2025
Merged

chore(ci): add fallback value for scheduled workflow run#11641
stormslowly merged 1 commit intomainfrom
chore/fix_crontab_canary_workflow

Conversation

@stormslowly
Copy link
Contributor

@stormslowly stormslowly commented Sep 11, 2025

Summary

Schduled workflow run doesn't have inputs.
So fallback values should be provided

Verified by forked repo https://github.com/stormslowly/rspack/actions/runs/17633394874

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings September 11, 2025 05:15
@netlify
Copy link

netlify bot commented Sep 11, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit c3fb57c
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/68c25b014d5a900008194fd2

@github-actions github-actions bot added release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack. labels Sep 11, 2025
Copy link
Contributor

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.

Pull Request Overview

This PR fixes the release-canary.yml workflow to handle scheduled workflow runs by adding fallback values for inputs that are undefined when the workflow is triggered by a cron schedule rather than manual dispatch.

  • Add fallback values for commit, profile, and test inputs in the reusable build workflow call
  • Ensure the workflow works correctly for both scheduled runs and manual dispatch
Comments suppressed due to low confidence (1)

.github/workflows/release-canary.yml:1

  • The ref parameter in the checkout step also needs a fallback value to handle scheduled workflow runs. It should use ${{inputs.commit || github.sha}} to be consistent with the build job.
name: Release Canary

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@stormslowly stormslowly changed the title fix: add fallback value for schedule workflow run chore(ci): add fallback value for schedule workflow run Sep 11, 2025
@github-actions github-actions bot removed the release: bug fix release: bug related release(mr only) label Sep 11, 2025
Copy link
Member

@chenjiahan chenjiahan left a comment

Choose a reason for hiding this comment

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

LGTM

@chenjiahan chenjiahan enabled auto-merge (squash) September 11, 2025 05:18
@stormslowly stormslowly enabled auto-merge (squash) September 11, 2025 05:19
@stormslowly stormslowly changed the title chore(ci): add fallback value for schedule workflow run chore(ci): add fallback value for scheduled workflow run Sep 11, 2025
@github-actions
Copy link
Contributor

📦 Binary Size-limit

Comparing c3fb57c to fix: persistent cache update issuer is updated modules (#11633) by jinrui

🙈 Size remains the same at 47.23MB

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 11, 2025

CodSpeed Performance Report

Merging #11641 will not alter performance

Comparing chore/fix_crontab_canary_workflow (c3fb57c) with main (d005492)

Summary

✅ 17 untouched benchmarks

@stormslowly stormslowly merged commit 41cf73d into main Sep 11, 2025
75 of 79 checks passed
@stormslowly stormslowly deleted the chore/fix_crontab_canary_workflow branch September 11, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants