Conversation
There was a problem hiding this comment.
Pull request overview
Adds a Laravel Workbench/Testbench setup to this package so contributors can boot and exercise the EuPago integration directly from a clone (tinker, migrate, serve) without creating a separate host Laravel application.
Changes:
- Adds Workbench scaffold under
workbench/(routes, minimal app structure, model/factory/seeder, env example). - Introduces
testbench.yamlplus Composer scripts/autoload-dev entries to support Workbench discovery/build/serve flows. - Updates contributor docs and gitignore to document the workflow and prevent accidental credential commits.
Reviewed changes
Copilot reviewed 11 out of 17 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| workbench/routes/web.php | Adds a default web route for the Workbench app. |
| workbench/routes/console.php | Adds a console routes stub for the Workbench app. |
| workbench/resources/views/.gitkeep | Tracks the Workbench views directory. |
| workbench/database/seeders/DatabaseSeeder.php | Adds a sample seeder for the Workbench app database. |
| workbench/database/migrations/.gitkeep | Tracks the Workbench migrations directory. |
| workbench/database/factories/UserFactory.php | Adds a sample User factory for Workbench. |
| workbench/database/factories/.gitkeep | Tracks the Workbench factories directory. |
| workbench/bootstrap/.gitkeep | Tracks the Workbench bootstrap directory. |
| workbench/app/Providers/WorkbenchServiceProvider.php | Adds a stub service provider for Workbench customization. |
| workbench/app/Models/User.php | Adds a sample User model for Workbench. |
| workbench/app/Models/.gitkeep | Tracks the Workbench models directory. |
| workbench/.env.example | Documents required Eupago sandbox env keys for local Workbench usage. |
| testbench.yaml | Configures Testbench/Workbench bootstrapping and build steps. |
| CONTRIBUTING.md | Documents the local Workbench-based workflow (tinker/migrate/serve). |
| composer.json | Adds Workbench autoload-dev namespaces and Composer scripts for discovery/build/serve. |
| .gitignore | Ignores .env files to reduce risk of committing credentials. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #73.
Enables testing the package against the real Eupago sandbox directly from a clone — no separate host application needed.
After comparing ecosystem conventions (laravel/passport commits a full Workbench scaffold; spatie/laravel-data and filament commit only a
testbench.yaml), this takes the minimal route:testbench.yamlregistersEuPagoServiceProviderand the package migrations path, so the Testbench CLI boots with the package loaded:vendor/bin/testbench tinker/migrate/serve..env(read by the Testbench CLI, never committed);.env.exampledocuments the expected keys..gitignorenow covers.envfiles — previously a root.envwith real credentials was onegit add .away from being committed.No
workbench/folder, nocomposer.jsonchanges — a full Workbench scaffold can be added later with one command if a demo app is ever needed.Verified locally:
config:show eupagoresolves the package config with.envvalues, tinker instantiates package classes,migrateruns all five package migrations, and the full quality suite passes (Pest, Pint, PHPStan).