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

fix and improves l10n #1181

Merged
merged 2 commits into from
Nov 25, 2023
Merged

fix and improves l10n #1181

merged 2 commits into from
Nov 25, 2023

Conversation

ijklam
Copy link
Contributor

@ijklam ijklam commented Oct 22, 2023

WHAT

🤖 Generated by Copilot at 26bce28

This pull request adds support for localized tooltips and code fixes for F# keywords, operators, errors, and warnings. It introduces new helper functions and modules to find and load localized XML files and match diagnostic messages. It modifies existing code fix modules to use the new functions and handle different message formats. It affects the files TipFormatter.fs, ResolveNamespace.fs, CodeFixes.fs, CodeFixes.fsi, and ReplaceWithSuggestion.fs.

🤖 Generated by Copilot at 26bce28

We're sailing on the F# sea, with code fixes for you and me
We match the messages by check, and handle colors with a regex
Heave away, me hearties, heave away
We'll make the code work in any tongue, with ifDiagnosticByCheckMessage and findLocalizedXmlFile

🌐🔧💡

WHY

  1. fix: codefix ReplaceWithSuggestion and ResolveNamespace can now deal with localized compiler texts
  2. improves: now can load localized xml documents

HOW

🤖 Generated by Copilot at 26bce28

  • Add support for localized tooltips for F# keywords and operators (link, link)
  • Add support for localized code fixes for F# errors and warnings (link, link, link, link, link)

fix: codefix ReplaceWithSuggestion and ResolveNamespace can now
deal with localized compiler texts
improves: now can load localized xml documents
Copy link
Member

@TheAngryByrd TheAngryByrd left a comment

Choose a reason for hiding this comment

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

This is a another good step in the right direction, thanks!

@baronfel anything you'd like to add?

@baronfel
Copy link
Contributor

IMO we can take this for now - but I think the proper fix would be for compiler changes to dump the relevant data into the diagnostic extra data, like some folks have started doing recently.

@baronfel
Copy link
Contributor

@Tangent-90 thank you for this contribution!

@baronfel baronfel merged commit 2a00261 into ionide:main Nov 25, 2023
8 of 9 checks passed
1eyewonder pushed a commit to 1eyewonder/FsAutoComplete that referenced this pull request Dec 2, 2023
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