Skip to content

Conversation

@mhdirkse
Copy link
Member

@mhdirkse mhdirkse commented Dec 4, 2025

No description provided.

@mhdirkse mhdirkse requested a review from jacodg December 4, 2025 15:04
@mhdirkse
Copy link
Member Author

mhdirkse commented Dec 4, 2025

Ik kan file ladybug-common/src/main/java/org/wearefrank/ladybug/Span.java niet goed checken. GitHub laat de diff niet duidelijk zien. Het lijkt dat alleen spaces door tabs zijn vervangen of omgekeerd, maar dat wordt niet getoond met donkerrood en donkergroen kleuren. @jacodg kan jij checken of dit OK is?

@mhdirkse
Copy link
Member Author

mhdirkse commented Dec 4, 2025

Ik heb de Echo2 Java files en de test code niet nagelopen. De rest wel.

@mhdirkse
Copy link
Member Author

mhdirkse commented Dec 4, 2025

Nu de GitHub Actions van dit project niet goed werken heb ik op de command line mvn clean install gedaan. Dat werkt.

@mhdirkse
Copy link
Member Author

mhdirkse commented Dec 4, 2025

De ladybug-frontend Cypress testen slagen met de backend van deze PR.

@mhdirkse mhdirkse marked this pull request as draft December 18, 2025 10:51
@mhdirkse
Copy link
Member Author

These are very many changes. It may be wise to use another .editorconfig that has less rules. This PR may be used as an example but it should not be merged as-is.

@mhdirkse
Copy link
Member Author

Dit is verouderd en op deze manier wordt de commit ook veel te groot. En we hebben wel urgentere issues dan formatting.

@mhdirkse mhdirkse closed this Jan 20, 2026
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