Skip to content

feat: install Event, Storage facades#1194

Merged
hwbrzzl merged 1 commit intomasterfrom
bowen/install-event
Sep 14, 2025
Merged

feat: install Event, Storage facades#1194
hwbrzzl merged 1 commit intomasterfrom
bowen/install-event

Conversation

@hwbrzzl
Copy link
Contributor

@hwbrzzl hwbrzzl commented Sep 13, 2025

📑 Description

Relate goravel/goravel#612

image image

✅ Checks

  • Added test cases for my code

@hwbrzzl hwbrzzl marked this pull request as ready for review September 13, 2025 15:33
@hwbrzzl hwbrzzl requested a review from a team as a code owner September 13, 2025 15:33
Copilot AI review requested due to automatic review settings September 13, 2025 15:33
Copy link
Contributor

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

This PR adds facade installation capabilities for Event and Storage modules in the Goravel framework. It enables automatic generation of facade files and configuration when these modules are installed.

  • Adds stub generation functionality for Event and Storage facades
  • Implements conditional facade file management during install/uninstall operations
  • Creates filesystem configuration stub for Storage module

Reviewed Changes

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

File Description
filesystem/setup/stubs.go Adds stub generators for Storage facade and filesystem configuration
filesystem/setup/setup.go Updates setup to use stubs and handle Storage facade installation/removal
event/setup/stubs.go Adds stub generator for Event facade
event/setup/setup.go Updates setup to use stubs and handle Event facade installation/removal

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Sep 13, 2025

Codecov Report

❌ Patch coverage is 0% with 82 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.15%. Comparing base (95fdf0c) to head (38ac935).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
filesystem/setup/stubs.go 0.00% 52 Missing ⚠️
event/setup/stubs.go 0.00% 11 Missing ⚠️
filesystem/setup/setup.go 0.00% 10 Missing ⚠️
event/setup/setup.go 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1194      +/-   ##
==========================================
- Coverage   69.49%   69.15%   -0.34%     
==========================================
  Files         229      231       +2     
  Lines       14391    14461      +70     
==========================================
  Hits        10001    10001              
- Misses       4032     4102      +70     
  Partials      358      358              

☔ 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.

@hwbrzzl hwbrzzl merged commit 6c1983e into master Sep 14, 2025
12 of 14 checks passed
@hwbrzzl hwbrzzl deleted the bowen/install-event branch September 14, 2025 00:13
alfanzain pushed a commit to alfanzain/goravel-framework that referenced this pull request Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants