Skip to content

CI: Remove deps scanner, switch to ubuntu runner#33854

Merged
alexslavr merged 7 commits into
26_1from
lavrov/publish-from-ubuntu-runner
Jun 5, 2026
Merged

CI: Remove deps scanner, switch to ubuntu runner#33854
alexslavr merged 7 commits into
26_1from
lavrov/publish-from-ubuntu-runner

Conversation

@alexslavr
Copy link
Copy Markdown
Contributor

@alexslavr alexslavr commented Jun 5, 2026

@alexslavr alexslavr self-assigned this Jun 5, 2026
Copilot AI review requested due to automatic review settings June 5, 2026 07:49
@alexslavr alexslavr requested a review from a team as a code owner June 5, 2026 07:49
@alexslavr alexslavr added the 26_1 label Jun 5, 2026
Copy link
Copy Markdown
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 updates the GitHub Packages publishing workflow to run on Linux runners and removes the dependency-scanner artifact from the build outputs, aligning CI artifacts with the new workflow behavior.

Changes:

  • Switched the build job in packages_publishing.yml from windows-latest to ubuntu-latest.
  • Removed the deps-scanner clone/build/run steps from the publishing workflow.
  • Removed DEPS_SCANNER_ARTIFACTS from artifact path constants and from the artifacts package assembly script.

Reviewed changes

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

File Description
tools/scripts/make-artifacts-package.ts Stops bundling the deps-scanner artifacts into devextreme-artifacts.
tools/scripts/common/paths.ts Removes the DEPS_SCANNER_ARTIFACTS path constant.
.github/workflows/packages_publishing.yml Switches CI build runner to Ubuntu and drops deps-scanner execution from the pipeline.
Comments suppressed due to low confidence (1)

.github/workflows/packages_publishing.yml:60

  • After switching the build job to ubuntu-latest, this step still uses PowerShell environment variable syntax ($env:NODE_AUTH_TOKEN) and a trailing ;. On Ubuntu the default shell is bash, so this will fail and prevent SBOM generation.
      - name: Build SBOMs
        env:
          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        run: |
          pnpm set "//npm.pkg.github.com/:_authToken" "$env:NODE_AUTH_TOKEN"
          pnpm nx build sbom;

Copilot AI review requested due to automatic review settings June 5, 2026 08:44
@alexslavr alexslavr force-pushed the lavrov/publish-from-ubuntu-runner branch from bfeab05 to 9c6521e Compare June 5, 2026 08:45
Copy link
Copy Markdown
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/packages_publishing.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Alex Lavrov <36633600+alexslavr@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 5, 2026 08:48
Copy link
Copy Markdown
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

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

@alexslavr alexslavr merged commit 265111c into 26_1 Jun 5, 2026
115 checks passed
@alexslavr alexslavr deleted the lavrov/publish-from-ubuntu-runner branch June 5, 2026 10:16
mpreyskurantov added a commit that referenced this pull request Jun 5, 2026
mpreyskurantov added a commit that referenced this pull request Jun 5, 2026
Signed-off-by: Mikhail Preyskurantov <5574159+mpreyskurantov@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants