Skip to content

ci: add workflow to test in goravel/example#1439

Merged
hwbrzzl merged 7 commits into
masterfrom
bowen/add-example-ci
Apr 11, 2026
Merged

ci: add workflow to test in goravel/example#1439
hwbrzzl merged 7 commits into
masterfrom
bowen/add-example-ci

Conversation

@hwbrzzl

@hwbrzzl hwbrzzl commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a new CI workflow that checks out goravel/example alongside the framework and runs its full test suite
  • Replace the example's framework dependency with the local checkout so pull requests are validated end-to-end
  • Run the tests against multiple Go versions in a matrix

Why

The framework currently has no automated check that verifies changes don't break the companion goravel/example application. Breakage is only discovered after a PR is merged, requiring a separate fix cycle.

This workflow triggers on every push to master and on every pull request. It checks out both goravel/framework and goravel/example, rewires the example's go.mod to point at the local framework tree, and executes go test ./... inside the example. This gives maintainers confidence that in-flight framework changes stay compatible with the canonical application scaffold.

# .github/workflows/test_example.yml
- name: Replace framework with local checkout
  working-directory: example
  run: |
    go mod edit -replace github.com/goravel/framework=../framework
    go mod tidy

- name: Run tests
  working-directory: example
  run: go test -timeout 10m -p 1 ./...

Copilot AI review requested due to automatic review settings April 11, 2026 02:12
@hwbrzzl
hwbrzzl requested a review from a team as a code owner April 11, 2026 02:12
Comment thread .github/workflows/test_example.yml Fixed
…ntain permissions'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

Copilot AI 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.

Pull request overview

Adds a new GitHub Actions workflow to run the full test suite of goravel/example against the framework code from the current commit/PR by checking out both repositories and using a local go.mod replace. This extends CI coverage to catch framework changes that break the canonical example app.

Changes:

  • Introduce .github/workflows/test_example.yml to checkout goravel/framework + goravel/example and run go test ./... in the example.
  • Rewire the example’s go.mod to use the local framework checkout via go mod edit -replace + go mod tidy.
  • Run the example test suite in a Go version matrix (1.25 / 1.26) with Go build/module caching.

Comment thread .github/workflows/test_example.yml Outdated
Comment thread .github/workflows/test_example.yml
Comment thread .github/workflows/test_example.yml Outdated
@codecov

codecov Bot commented Apr 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.78%. Comparing base (22dc71f) to head (940f3d7).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1439   +/-   ##
=======================================
  Coverage   68.78%   68.78%           
=======================================
  Files         361      361           
  Lines       27833    27833           
=======================================
  Hits        19145    19145           
  Misses       7840     7840           
  Partials      848      848           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 11, 2026 02:20

Copilot AI 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.

Pull request overview

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

Comment thread .github/workflows/test_example.yml
Comment thread .github/workflows/test_example.yml Outdated
Comment thread .github/workflows/test_example.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 11, 2026 02:26

Copilot AI 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.

Pull request overview

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

Comment thread .github/workflows/test_example.yml
Comment thread .github/workflows/test_example.yml Outdated
Comment thread .github/workflows/test_example.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 11, 2026 02:30

Copilot AI 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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/test_example.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 11, 2026 02:35

Copilot AI 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.

Pull request overview

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

@hwbrzzl
hwbrzzl merged commit f08b9a2 into master Apr 11, 2026
23 checks passed
@hwbrzzl
hwbrzzl deleted the bowen/add-example-ci branch April 11, 2026 02:46
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