Skip to content
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 tests #606

Merged
merged 2 commits into from
May 31, 2024
Merged

Fix tests #606

merged 2 commits into from
May 31, 2024

Conversation

arthurkirkosa
Copy link
Collaborator

No description provided.

@arthurkirkosa arthurkirkosa requested a review from binaryk May 31, 2024 05:29
Copy link

what-the-diff bot commented May 31, 2024

PR Summary

  • PHP Version Update in Workflow
    The work sequence within '.github/workflows/psalm.yml' has been adjusted to incorporate a newer version of PHP. This indicates that our underlying programming environment is getting an upgrade for better performance and enhanced features.

  • Code Refactoring in AuthorizableModels.php
    The code in 'src/Traits/AuthorizableModels.php' has been restructured and streamlined. A part of this refactoring includes changing the way messages are written, now using multiline strings. This makes our code easier to read and maintain.

  • New Configuration in Test Suite
    A fresh configuration has been added within the 'tests/Feature/AuthorizableModelsTest.php' portion. This contributes to an expanded and more accurate testing environment, ensuring the reliability of our code.

  • Parameter Addition in Namespace Test Function
    Within 'tests/Unit/RepositoriesResolvedFromNamespaceTest.php', an additional parameter was introduced to the 'test_repository_can_be_resolved_from_any_namespace' function. This allows us to test our repository handling more extensively, further ensuring the robustness of our system.

@arthurkirkosa arthurkirkosa merged commit 1c29d53 into 9.x May 31, 2024
9 checks passed
@arthurkirkosa arthurkirkosa deleted the fix-tests branch May 31, 2024 14:04
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.

2 participants