Skip to content

Conversation

@kewisch
Copy link
Member

@kewisch kewisch commented May 16, 2025

Related to #9122

Copy link
Member

@wmontwe wmontwe left a comment

Choose a reason for hiding this comment

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

LGTM

@wmontwe
Copy link
Member

wmontwe commented May 19, 2025

Fixed the badging information

@wmontwe
Copy link
Member

wmontwe commented May 20, 2025

The build is failing because of an old issue with Spotless, that is fixed on main and beta already.

I've run the build locally and it succeeds. Either we merge this as is or we need to backport the Spotless fix first.

@kewisch
Copy link
Member Author

kewisch commented May 20, 2025

Is the spotless failure uncovered through these changes in some way? I think the only think we need to avoid is that this spotless failure is going to make the release builds fail. If that isn't the case (e.g. it worked for the last release build which should have the same spotless errors), then we could merge.

@wmontwe
Copy link
Member

wmontwe commented May 20, 2025

It's a combination of Gradle cache and Spotless, it might happen for release too.

@kewisch
Copy link
Member Author

kewisch commented May 20, 2025

If there is a chance it might succeed, I guess we can always backport the spotless fix if the release fails.

@kewisch kewisch merged commit c940734 into thunderbird:release May 20, 2025
2 of 6 checks passed
@kewisch kewisch deleted the release-fix-changelog branch May 20, 2025 10:42
@kewisch kewisch added this to the Thunderbird 10 milestone May 21, 2025
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.

3 participants