Skip to content

Fix rust analyzer error when translation file change#157

Open
wiiznokes wants to merge 2 commits intokellpossible:masterfrom
wiiznokes:cache-invalidation-test
Open

Fix rust analyzer error when translation file change#157
wiiznokes wants to merge 2 commits intokellpossible:masterfrom
wiiznokes:cache-invalidation-test

Conversation

@wiiznokes
Copy link
Copy Markdown

fix #147

Tho, i'm not sure this is the best solution. I think a better one would be to use the build.rs file. cargo-i18n could provide an helper for this

@wiiznokes wiiznokes changed the title Cache invalidation test Cache invalidation Aug 18, 2025
@wiiznokes wiiznokes changed the title Cache invalidation Fix rust analyzer error when translation file change Aug 18, 2025
@wiiznokes
Copy link
Copy Markdown
Author

Using a build.rs in i18n-embed-fl would probably work, but there is the problem of retreiving the Manifest dir. We can't use the MANIFEST_DIR env variable, and using a custom env variable will probably not work when multiple crate are using i18n-embed-fl

@wiiznokes wiiznokes marked this pull request as ready for review August 19, 2025 11:40
@wiiznokes
Copy link
Copy Markdown
Author

technicallty, we could use a custom env variable as a list of path. This might be the more elegant solution

@kellpossible
Copy link
Copy Markdown
Owner

@wiiznokes thanks for contribution! Currently a bit snowed under with other life things but I'll review as soon as I get the chance

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.

rust-analyzer currently broken when adding a new translation

2 participants