Link to issue number:

None; related to #13333.

Summary of the issue:

The file th\key commands.txt was an old outdated that was removed during the migration to Crowdin. Now the local documentations do not contain any old txt file, but getDocFilePath had still some unused code to handle these files.

Description of user facing changes

None.

Description of development approach

Remove dead code in both getDocFilePath functions.

While at it, updated .git-ignore to also ignore all the .md files that are generated when building the documentation (scons user_docs), since the source strings are now in .xliff files.

Testing strategy:

• Test opening the documentation (user guide, key commands and change log) from NVDA menu
• From Python console, test installer.getDocFilePath with the same files.
• Test English, de and de_CH, trying to manually remove some documentation to simulate fallback or using other languages where documentation is missing (e.g. "am", "nb_NO")
• Try to open licence file from installer (runnvda --launcher) and from NVDA menu

Known issues with pull request:

None

Code Review Checklist:

• Documentation:
  • Change log entry
  • User Documentation
  • Developer / Technical Documentation
  • Context sensitive help for GUI changes
• Testing:
  • Unit tests
  • System (end to end) tests
  • Manual testing
• UX of all users considered:
  • Speech
  • Braille
  • Low Vision
  • Different web browsers
  • Localization in other languages / culture than English
• API is compatible with existing add-ons.
• Security precautions taken.

@coderabbitai summary