Skip to content

Add Dependabot workflow to rebuild dist#15

Open
digimangos wants to merge 8 commits into
mainfrom
digimangos/fix-dependabot-dist
Open

Add Dependabot workflow to rebuild dist#15
digimangos wants to merge 8 commits into
mainfrom
digimangos/fix-dependabot-dist

Conversation

@digimangos

Copy link
Copy Markdown
Collaborator

This pull request introduces a new GitHub Actions workflow to automatically rebuild and commit the dist directory when Dependabot opens or updates pull requests that modify package management files. This helps ensure that distribution files are always up-to-date after dependency updates from Dependabot.

@digimangos digimangos requested a review from sparlant as a code owner June 4, 2026 16:43
Copilot AI review requested due to automatic review settings June 4, 2026 16:43

Copilot AI left a comment

Copy link
Copy Markdown

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 adds a GitHub Actions workflow intended to automatically rebuild and commit generated distribution artifacts (dist) when Dependabot opens or updates dependency-related pull requests.

Changes:

  • Introduces a new Dependabot Rebuild Dist workflow triggered on Dependabot PRs that touch npm package management files.
  • Runs a Node install/build and pushes updated dist output back to the Dependabot branch.
Show a summary per file
File Description
.github/workflows/dependabot-rebuild-dist.yml Adds a workflow to rebuild and commit dist artifacts for Dependabot PRs.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 5

Comment thread .github/workflows/dependabot-rebuild-dist.yml Outdated
Comment thread .github/workflows/dependabot-rebuild-dist.yml Outdated
Comment thread .github/workflows/dependabot-rebuild-dist.yml Outdated
Comment thread .github/workflows/dependabot-rebuild-dist.yml
Comment thread .github/workflows/dependabot-rebuild-dist.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 4, 2026 16:49
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
digimangos and others added 3 commits June 4, 2026 17:49
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@digimangos

Copy link
Copy Markdown
Collaborator Author

@copilot review this PR again

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 3

Comment thread .github/workflows/dependabot-rebuild-dist.yml
Comment thread .github/workflows/dependabot-rebuild-dist.yml Outdated
Comment thread .github/workflows/dependabot-rebuild-dist.yml

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown

@copilot review this PR again

Addressed with commit 87b3ad3.

I made a minimal follow-up update to the Dependabot workflow: scoped pull_request.paths to issueops-common-logic/*package* files and removed the unused pull-requests: read permission. I also re-ran tests (npm run test) and they pass.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 3

Comment on lines +27 to +29
with:
ref: ${{ github.event.pull_request.head.ref }}
fetch-depth: 0
Comment on lines +34 to +36
node-version-file: .node-version
cache: npm
cache-dependency-path: issueops-common-logic/package-lock.json
Comment on lines +39 to +44
run: |
if [ -f package-lock.json ] || [ -f npm-shrinkwrap.json ]; then
npm ci
else
npm install
fi
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.

3 participants