Skip to content

Setup github actions ci and docker ci - #5

Merged
Toltar merged 7 commits into
developmentfrom
1-github-actions-ci-pipeline
Jul 9, 2026
Merged

Setup github actions ci and docker ci#5
Toltar merged 7 commits into
developmentfrom
1-github-actions-ci-pipeline

Conversation

@Toltar

@Toltar Toltar commented Jul 2, 2026

Copy link
Copy Markdown

This PR:

Resolves #1

Implements GitHub actions with parallel builds for the application and the docker build on PRs and merges.

@vercel

vercel Bot commented Jul 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pdxhub Ready Ready Preview, Comment Jul 9, 2026 1:47am

@Toltar
Toltar requested a review from Jared-Krajewski July 2, 2026 20:11
@Toltar Toltar self-assigned this Jul 2, 2026

@Jared-Krajewski Jared-Krajewski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A couple small things commented on the files. Thanks for getting this going.

Comment thread .github/workflows/actionslint.yml Outdated
Comment on lines +11 to +14
strategy:
matrix:
os: [ubuntu-latest]
node_version: [24.x]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think anything here uses this node version and the os is otherwise isolated so we should be able to run this without the matrix and just runs-on: to ubuntu-latest.

this could reduce a little build overhead.

docs

Comment thread .github/workflows/ci.yml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we should add this to both files

permissions:
  contents: read

docs

Comment thread .github/workflows/ci.yml Outdated
Comment on lines +13 to +16
strategy:
matrix:
os: [ubuntu-latest]
runs-on: ${{ matrix.os }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same as other file

Comment thread .github/workflows/ci.yml Outdated
Comment on lines +25 to +28
strategy:
matrix:
os: [ubuntu-latest]
runs-on: ${{ matrix.os }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same as above

@Jared-Krajewski Jared-Krajewski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good! 🚀

@Toltar
Toltar merged commit 2f8a2da into development Jul 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] - Add GitHub Actions CI Pipeline (Build Check)

2 participants