- 
                Notifications
    
You must be signed in to change notification settings  - Fork 0
 
fix: adding middleware for telescope authorization #73
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.
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
fix: adding middleware for telescope authorization #73
Conversation
          Pull Request Test Coverage Report for Build 19060845829Details
 
 💛 - Coveralls | 
    
There was a problem hiding this 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 fixes the order of operations during project initialization by moving the telescope authorization middleware setup to run after shell commands are executed. This ensures that the telescope configuration file exists before the middleware configuration is applied.
- Moved the telescope authorization middleware setup to execute after shell commands
 - Reordered test expectations to match the new execution sequence
 
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description | 
|---|---|
| src/Commands/InitCommand.php | Moved telescope middleware configuration to execute after shell commands | 
| tests/InitCommandTest.php | Updated test expectations to match the new order of telescope configuration operations | 
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…are_for_telescope_authorization # Conflicts: # tests/InitCommandTest.php
refs: #38
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
refs: #38