Skip to content

Conversation

nckrtl
Copy link
Contributor

@nckrtl nckrtl commented Jun 4, 2025

This PR migrates from mix to Vite. I created a new PR as #1474 is open for already a year. Felt easier to start with a fresh slate and it allowed me to get a better feeling of the project structure.

I tried to mimic Horizon's project structure as much as possible. Copied the same stuff over like .gitattributes, style.ci etc. Also the way assets are handled are similar to Horizon, which is a remark in the other PR.

If there is a preference to update the other PR instead, I can give that a try as well.

@nckrtl
Copy link
Contributor Author

nckrtl commented Jun 4, 2025

Bit unsure about styleci. I'd expect it to commit style preferences, but it only reports them. Tried to fix them but its now reporting about an expected trailing comma at resources/css/styles-dark.scss while it was not expecting one for resources/css/styles.scss

@taylorotwell taylorotwell marked this pull request as draft June 4, 2025 13:18
@nckrtl nckrtl marked this pull request as ready for review June 14, 2025 10:00
@taylorotwell taylorotwell merged commit 252e915 into laravel:5.x Jun 18, 2025
16 checks passed
@taylorotwell
Copy link
Member

Thanks!

crynobone added a commit that referenced this pull request Aug 13, 2025
The tag has been removed via #1598

Signed-off-by: Mior Muhammad Zaki <[email protected]>
taylorotwell pushed a commit that referenced this pull request Aug 13, 2025
The tag has been removed via #1598

Signed-off-by: Mior Muhammad Zaki <[email protected]>
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