Skip to content

Conversation

@jmsche
Copy link
Contributor

@jmsche jmsche commented Aug 25, 2025

Q A
Bug fix? no
New feature? no
Tests pass? I guess
Fixed tickets N/A

I guess these were forgotten during cleanup? 😅

@Kocal
Copy link
Owner

Kocal commented Aug 26, 2025

Nope they are needed in the CI, they must be ignored in the .gitattributes instead

@jmsche
Copy link
Contributor Author

jmsche commented Aug 26, 2025

I haven't found any references to these files in the CI, and it seems the CI passes?

@Kocal
Copy link
Owner

Kocal commented Aug 26, 2025

Mmmh indeed, in E2E tests Biome runs on a new Symfony app and not fixtures file, they are purely internal.

Well, let's remove them, thanks!

PS: the same issue exists in https://github.com/Kocal/OxlintBundle if you want to become a contributor :p

@Kocal Kocal merged commit f5b4f72 into Kocal:main Aug 26, 2025
19 of 20 checks passed
@jmsche jmsche deleted the remove-unneeded-files branch August 26, 2025 11:22
@jmsche
Copy link
Contributor Author

jmsche commented Aug 26, 2025

I'll have a look this week :)

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