Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proposed v2.5.2 release #5188

Merged
merged 29 commits into from
Oct 11, 2024
Merged

Conversation

andydotxyz
Copy link
Member

@andydotxyz andydotxyz commented Oct 10, 2024

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

williambrode and others added 26 commits October 10, 2024 12:02
Currently translated at 100.0% (39 of 39 strings)

Translation: Fyne/Fyne
Translate-URL: https://hosted.weblate.org/projects/fyne/fyne/sv/
This mainly does two things:
- Uses Go 1.22 instead of 1.21 as the highest version (I don't have 1.23
  on Fedora 40 yet so did not want to bump to latest just yet).
- We now always pull the latest setup-go-faster action in v1 without
  having to manually change version.
This fixes a few code errors that slipped through review (thankfully now
handled by Staticcheck for us) and makes sure we are using the latest
version.
Haven't quite gotten used to Nvim yet ;)
This should be safer than ever given that some new features are
dependent on the version in go.mod.
…SetWindowAttribute and DWMWA_USE_IMMERSIVE_DARK_MODE on windows.
@andydotxyz andydotxyz changed the base branch from master to release/v2.5.x October 10, 2024 12:36
It was finding the child before changing to the desired directory!
Copy link
Member

@Jacalz Jacalz left a comment

Choose a reason for hiding this comment

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

Does this have the changes that I made for improving theme lookups on Linux? I couldn't see it in the changelog but it seems like something that would be useful.

CHANGELOG.md Outdated

### Fixed

* Sometimes fyne fails to setup dark mode in Windows (#4758)
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't it be "on Windows"?

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks, fixed

@andydotxyz
Copy link
Member Author

Does this have the changes that I made for improving theme lookups on Linux? I couldn't see it in the changelog but it seems like something that would be useful.

Wasn't that resolved in 2.5.1?

@Jacalz
Copy link
Member

Jacalz commented Oct 11, 2024

Ah, right. Sorry 😅

@andydotxyz andydotxyz merged commit 040623c into fyne-io:release/v2.5.x Oct 11, 2024
10 of 11 checks passed
@andydotxyz andydotxyz deleted the release/v2.5.2 branch October 11, 2024 07:09
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.

8 participants