Skip to content

Conversation

@ictbeheer
Copy link
Contributor

This pull request introduces a rebranding and major refactor of the package, renaming it from "Sage Child Theme Support" to "Nutshell" and enhancing its functionality for WordPress theme development. Key changes include namespace updates, improved documentation, and additional features such as Vite asset support and Sentry integration.

Rebranding and Namespace Updates:

  • Updated the package name to "Nutshell" and changed all namespaces from Yard\SageChildThemeSupport to Yard\Nutshell across the codebase. This includes files like src/Assets/Vite.php, src/Bootstrap/LoadConfiguration.php, and others. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Documentation Enhancements:

  • Overhauled the README.md to reflect the new branding and features, including a detailed explanation of Nutshell's capabilities such as child theme configuration inheritance, Vite integration, Sentry support, and strict code quality workflows.

Composer Configuration:

  • Updated the composer.json to reflect the new namespace Yard\Nutshell for PSR-4 autoloading.

Feature Enhancements:

  • Introduced support for modern asset bundling with Vite, seamless Sentry error reporting, manual registration of view composers, and custom Artisan commands. These features are documented in the updated README.md.

Helper Function Updates:

  • Updated the bootloader() helper function to bind the new Nutshell classes for configuration, console kernel, exception handling, and asset management.

> After this change:
>
> - View Composers in the app/View/Composers directory will no longer be loaded automatically. To ensure they are registered, you have to configure them manually.
> - Console Commands in the app/Console/Commands directory will no longer be loaded automatically. To ensure they are register, you have to configure them manually.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deze stappen 1 t/m 6 en de Important warning zou ik behouden.

@ictbeheer ictbeheer force-pushed the feat/rename branch 2 times, most recently from 72f5567 to e8c7439 Compare June 19, 2025 13:42
@ictbeheer ictbeheer linked an issue Jun 20, 2025 that may be closed by this pull request
@ictbeheer ictbeheer requested a review from YvetteNikolov June 24, 2025 08:38
@ictbeheer ictbeheer merged commit 522dac2 into main Jun 24, 2025
4 checks passed
@ictbeheer ictbeheer deleted the feat/rename branch June 24, 2025 11:06
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.

Rename to nutshell

4 participants