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

Dev #64

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Dev #64

wants to merge 21 commits into from

Conversation

dconco
Copy link
Member

@dconco dconco commented Jan 17, 2025

This pull request includes changes to the Router and Core components of the PhpSlides application, focusing on namespace refactoring, case sensitivity handling, and workflow improvements. The most important changes include namespace updates, case sensitivity handling in routing, and modifications to GitHub workflows.

Namespace Refactoring:

  • Updated namespaces from PhpSlides\Src to PhpSlides\Core across multiple files including Router/MapRoute.php, Router/Route.php, Router/view.php, src/Bootstrap/App.php, src/Cache/Cache.php, src/Config/config.php, and various controller files. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Case Sensitivity Handling:

  • Added case sensitivity handling in routing by introducing a caseSensitive method in Router/Route.php and updating the match and match_routing methods in Router/MapRoute.php to consider case sensitivity. [1] [2] [3]

GitHub Workflows:

  • Renamed workflow steps in .github/workflows/release.yml and .github/workflows/tests.yml for better clarity. [1] [2]
  • Modified the Push changes to main job in .github/workflows/tests.yml to use a different approach for merging and pushing changes.

Additional Changes:

  • Updated the composer.json and phpunit.xml files to reflect new directory structures and autoload configurations. [1] [2]
  • Changed methods in src/Cache/Cache.php to be static and updated the clear and clearHotReload methods accordingly. [1] [2]

…lides\Core and improve test script execution
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.

1 participant