Skip to content

Commit 1261d1c

Browse files
janechuCopilot
andcommitted
fix(ci): tidy azure-pipelines-cd review nits
- Replace the DST-fragile `1am PST = 09:00 UTC` cron comment with a UTC-anchored phrasing that also acknowledges the \~1am PT / \~2am PDT seasonal drift, so readers do not assume the schedule tracks Pacific time year-round. - Drop the unused `undeployedTags` stage variable from the Package stage. It was bound to the Check stage output but never consumed downstream — the Package stage's own `download-github-releases.mjs` invocation re-enumerates undeployed releases from the local git tag set, so removing the captured variable also removes the implicit expectation that the two stages share that list. The script still emits the output via `##vso[task.setvariable]` for visibility in the Check stage log. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 3ad3ca6 commit 1261d1c

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

azure-pipelines-cd.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ trigger: none
22
pr: none
33

44
schedules:
5-
- cron: '0 9 * * *' # 1am PST = 09:00 UTC.
5+
- cron: '0 9 * * *' # 09:00 UTC daily (~1am Pacific in standard time, ~2am during DST).
66
displayName: Daily npm/crates publish from GitHub Releases
77
branches:
88
include:
@@ -79,7 +79,6 @@ extends:
7979
condition: eq(dependencies.Check.outputs['CheckVersion.deploymentCheck.needsDeployment'], 'true')
8080
variables:
8181
npm_config_cache: $(Pipeline.Workspace)/.npm
82-
undeployedTags: $[ stageDependencies.Check.CheckVersion.outputs['deploymentCheck.undeployedTags'] ]
8382
jobs:
8483
- job: Deploy
8584
steps:

0 commit comments

Comments
 (0)