Skip to content

Skip npm ci call in dependabot bundler job#475

Merged
JC-DSIT merged 1 commit into
mainfrom
workflows/fix-bundler-dependabot-job
Apr 22, 2026
Merged

Skip npm ci call in dependabot bundler job#475
JC-DSIT merged 1 commit into
mainfrom
workflows/fix-bundler-dependabot-job

Conversation

@JC-DSIT
Copy link
Copy Markdown
Contributor

@JC-DSIT JC-DSIT commented Apr 22, 2026

Proposed changes

What changed

This is a small change to allow the dependabot job to run against the bundler

if ENV["DEPENDABOT_JOB"] == "true" # dependabot will only evaluate the bundler, and will not allow npm
  warn "Skipping npm ci due to DEPENDABOT_JOB=true"
else
  system("npm ci")
  abort "npm ci failed" unless $CHILD_STATUS.success?
end

Related issue or tracking reference

You should have an open GitHub issue that this PR will fix.

Checklist

Before you request approval you should confirm that:

  • the pull request has a clear title with a short description about the update in the documentation
  • GitHub actions all pass
  • you have tested the changes with a fresh build against the latest version of the tech-docs-gem
  • you have linked this PR to an issue (or explained why none exists)
  • you have updated documentation if needed

@JC-DSIT JC-DSIT merged commit 6ad6cac into main Apr 22, 2026
4 checks passed
@JC-DSIT JC-DSIT deleted the workflows/fix-bundler-dependabot-job branch April 22, 2026 15:18
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.

1 participant