Skip to content

feat: add CI workflow for linting and building with Bun - #21

Open
staciax wants to merge 2 commits into
devfrom
feat/add-ci-github-actions
Open

feat: add CI workflow for linting and building with Bun#21
staciax wants to merge 2 commits into
devfrom
feat/add-ci-github-actions

Conversation

@staciax

@staciax staciax commented Jun 24, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@staciax
staciax requested a review from Copilot June 24, 2025 15:59

This comment was marked as outdated.

@staciax
staciax requested a review from Copilot July 2, 2025 09:50

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 introduces a new GitHub Actions workflow to set up Bun, install dependencies, run linting, and build the project, and pins the Bun version.

  • Add CI workflow (.github/workflows/ci.yaml) for Bun setup, linting, and build
  • Pin Bun version to 1.2.17 in .bun-version

Reviewed Changes

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

File Description
.github/workflows/ci.yaml Create CI job to checkout, setup Bun, install, lint, and build
.bun-version Specify Bun version (1.2.17) for the workflow
Comments suppressed due to low confidence (3)

.github/workflows/ci.yaml:10

  • The workflow is configured to trigger pull requests only against the master branch, but the repository’s default branch appears to be main. Update the branches setting to main so CI runs on PRs.
      - master

.github/workflows/ci.yaml:34

  • The test step is commented out, so tests won’t run in CI. Consider enabling this step or removing it if tests aren’t ready to ensure test coverage is validated on every run.
      # - name: Run tests

.github/workflows/ci.yaml:17

  • [nitpick] The job ID lint also runs build steps, which may be misleading. Consider renaming it to ci or splitting lint and build into separate jobs for clearer intent.
  lint:

@staciax
staciax force-pushed the dev branch 2 times, most recently from bebb897 to db0bbc4 Compare July 2, 2025 12:43
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