Skip to content

Add in GitHub Actions #48

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
FyreByrd merged 40 commits into from
Nov 18, 2025
Merged

Add in GitHub Actions #48

FyreByrd merged 40 commits into from
Nov 18, 2025

Conversation

@ben-laird
Copy link
Collaborator

@ben-laird ben-laird commented Nov 17, 2025

This PR adds CI/CD workflows via GitHub Actions. The CI action runs tests, runs checks, and lints.

@ben-laird
ci: add initial ci scripts
bb1bf8c 
Add test & preflight CI scripts to test that I'm doing this right lol
@ben-laird
ci: add repo checkout steps (duh)
4bb8397
@ben-laird
ci: add dependency caching, polish other steps
39b7c24
@ben-laird
ci: add fn-setup reusable workflow
31a9983 
Add fn-setup to encapsulate all setup steps and DRY up the main ci workflow
@ben-laird
ci: debug deno directory
84a3dce
@ben-laird
ci: test out whether specifying DENO_DIR makes caching work properly
00aae3a
@ben-laird
ci: add DENO_DIR env variable to Set up Deno action
4599439
@ben-laird
ci: switch up debugging, remove DENO_DIR
36f608d
@ben-laird
ci: change debug command
fa9d203
@ben-laird
ci: try using cache dir from cache action docs
0c1e197
@ben-laird
ci: update setup fn workflow with fixes from ci workflow
28d3fcd
@ben-laird
ci: refactor ci workflow to use setup fn workflow and run tests
2b4d0c1
@ben-laird
ci: move checkout action to outside workflow fn
b989437
@ben-laird
ci: fix using workflow fn
7c8a608
@ben-laird
ci: remove preflight
0186cd5
@ben-laird
ci: rename CI workflow to proper file extension
5191bf3
@ben-laird
ci: rename setup fn workflow to proper file extension
7a82cae
@ben-laird
ci: create fn-setup composite workflow
fff4dbd
@ben-laird
ci: try using composite workflow instead of reusable workflow
b4ebb94
@ben-laird
ci: supply paths to deno deps cache action
6da686d
@ben-laird
ci: clean up naming
35b1746
@ben-laird
ci: debug: what happens when we remove the deps cache
58604d5
@ben-laird
ci: debug: now what happens if we take away the cache hash
4bd3d87
@ben-laird
ci: simplify everything back down to a single CI workflow
136600f
@ben-laird
ci: be sure to check out repository again
ae2c41f
@ben-laird
ci: debug: what happens when we remove deps cache
0a62ace
@ben-laird
ci: debug: what happens when we take out explicit caching and only sp…
3e2132c 
…ecify a hash
@ben-laird
ci: debug: what happens when we use it for both
712ad90
@ben-laird
ci: officially remove vestigial caching
6e787e1
@ben-laird
ci: debug: what happens when we disable caching for test job
084c16e
@ben-laird ben-laird marked this pull request as ready for review November 17, 2025 20:46
@ben-laird ben-laird requested a review from FyreByrd November 17, 2025 20:47
@ben-laird ben-laird marked this pull request as draft November 17, 2025 20:48
@ben-laird ben-laird changed the base branch from main to dev November 17, 2025 20:49
@ben-laird ben-laird marked this pull request as ready for review November 17, 2025 20:49
FyreByrd
FyreByrd approved these changes Nov 18, 2025
View reviewed changes
Copy link
Collaborator

@FyreByrd FyreByrd left a comment

Choose a reason for hiding this comment

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

Looks good to me

@FyreByrd FyreByrd merged commit 026e0d3 into dev Nov 18, 2025
8 checks passed
@FyreByrd FyreByrd deleted the actions/litmus-test branch November 18, 2025 01:38
@ben-laird ben-laird linked an issue Nov 18, 2025 that may be closed by this pull request
Add CI/CD #12
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FyreByrd FyreByrd FyreByrd approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add CI/CD

3 participants

@ben-laird @FyreByrd