Skip to content

fix: report nightly build failures as Nightly, not Pre-release, in Slack - W-24680731 - #201

Merged
madhur310 merged 1 commit into
mainfrom
ms/W-24680731-nightly-slack-release-type
Oct 27, 2026
Merged

madhur310 merged 1 commit into
mainfrom
ms/W-24680731-nightly-slack-release-type

Conversation

@madhur310

Copy link
Copy Markdown
Contributor

Summary

  • vscode-publish-extensions.yml's Slack success/failure notifications computed release_type purely from the pre-release input, with no "Nightly" option
  • Since nightly cron runs always pass pre-release: 'true', a routine nightly build failure showed up in Slack as release_type: "Pre-release" -- indistinguishable from an actual marketplace Pre-release promotion failing
  • Fix: capture inputs.nightly into an is_nightly step output and check it before pre_release when computing release_type, so nightly failures now report "Nightly"

Test plan

  • Trigger a dry-run of nightly.yml from forcedotcom/salesforcedx-vscode against this branch ref and confirm the dry-run log line shows Release Type: Nightly
  • Confirm release_type: "Pre-release" is unchanged for the actual weekly promote-to-prerelease.yml flow

W-24680731

The release_type sent to the Slack failure/success notification only ever
checked the pre-release input, so nightly cron runs (which always pass
pre-release: 'true') showed up as "Pre-release" failures -- indistinguishable
from an actual marketplace pre-release promotion breaking.

W-24680731
@madhur310
madhur310 requested a review from peternhale October 27, 2026 14:05
@madhur310
madhur310 merged commit 550cf37 into main Oct 27, 2026
4 checks passed
@peternhale
peternhale deleted the ms/W-24680731-nightly-slack-release-type branch October 27, 2026 15:01
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.

2 participants