Skip to content

chore: add CI workflow and Dependabot config#4

Open
100yenadmin wants to merge 1 commit into
mainfrom
chore/add-ci-dependabot
Open

chore: add CI workflow and Dependabot config#4
100yenadmin wants to merge 1 commit into
mainfrom
chore/add-ci-dependabot

Conversation

@100yenadmin
Copy link
Copy Markdown
Member

@100yenadmin 100yenadmin commented Apr 2, 2026

Part of 100yenadmin/electric-sheep#1827 and 100yenadmin/electric-sheep#1828

Adds:

  • CI: npm ci, typecheck, build on PRs and push to main
  • Dependabot: npm and github-actions weekly updates

Open with Devin

Copilot AI review requested due to automatic review settings April 2, 2026 17:22
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Copy link
Copy Markdown

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

Adds basic automation to the repo by introducing a GitHub Actions CI workflow for PRs/pushes to main, plus a Dependabot configuration to keep npm and GitHub Actions dependencies up to date (per 100yenadmin/electric-sheep#1827 and #1828).

Changes:

  • Add a CI workflow that runs npm ci, TypeScript typechecking, and npm run build on PRs and pushes to main.
  • Add Dependabot config for weekly npm and GitHub Actions update PRs (with a PR limit and labels).

Reviewed changes

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

File Description
.github/workflows/ci.yml New CI workflow for install + typecheck + build on PR/push with concurrency cancellation.
.github/dependabot.yml New Dependabot config for weekly npm and GitHub Actions updates.

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

Comment thread .github/workflows/ci.yml
Comment on lines +1 to +12
name: CI

on:
push:
branches: [main]
pull_request:
branches: [main]

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true

Copy link

Copilot AI Apr 2, 2026

Choose a reason for hiding this comment

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

Consider explicitly setting minimal permissions for the workflow/job (e.g., contents: read) since this CI only checks out code and runs builds. Relying on the repo default token permissions can unintentionally grant more access than needed.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml

- uses: actions/setup-node@v4
with:
node-version: "20"
Copy link

Copilot AI Apr 2, 2026

Choose a reason for hiding this comment

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

actions/setup-node supports built-in dependency caching (cache: 'npm' with cache-dependency-path: package-lock.json). Enabling it can significantly reduce CI runtime and network usage on repeated runs.

Suggested change
node-version: "20"
node-version: "20"
cache: 'npm'
cache-dependency-path: package-lock.json

Copilot uses AI. Check for mistakes.
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