Skip to content

feat: he4rt domain - #103

Merged
danielhe4rt merged 6 commits into
3.xfrom
feat/he4rt-components
Nov 12, 2025
Merged

feat: he4rt domain#103
danielhe4rt merged 6 commits into
3.xfrom
feat/he4rt-components

Conversation

@danielhe4rt

@danielhe4rt danielhe4rt commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

This pull request introduces a new Composer configuration for the he4rt-core library, updates font file paths in the CSS to reflect a new directory structure, and adds debugging to the Rector GitHub Actions workflow. The most significant changes are grouped below:

New module configuration:

  • Added a composer.json for the he4rt-core library in app-modules/he4rt, defining its autoloading, version, and Laravel service provider registration.

Asset path updates (directory restructuring):

  • Renamed app-modules/portal/resources/css/support/fonts.css to app-modules/he4rt/resources/css/support/fonts.css and updated all font file paths to point to the new relative location, ensuring correct asset loading after the move. [1] [2]

CI/CD improvements:

  • Updated the Rector step in .github/workflows/_rector.yml to include the --debug flag for more detailed output during static analysis.

@danielhe4rt danielhe4rt self-assigned this Nov 12, 2025
@DiogoKaster
DiogoKaster marked this pull request as ready for review November 12, 2025 19:46
DiogoKaster
DiogoKaster previously approved these changes Nov 12, 2025

@DiogoKaster DiogoKaster left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

- update otavio-araujo/filament-smart-cep to 4.0.1
Clintonrocha98
Clintonrocha98 previously approved these changes Nov 12, 2025
- update rector/rector to 2.2.7
- update phpstan/phpstan to 2.1.26
@danielhe4rt
danielhe4rt merged commit edc1184 into 3.x Nov 12, 2025
5 checks passed
@danielhe4rt
danielhe4rt deleted the feat/he4rt-components branch November 12, 2025 21:45
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.

5 participants