Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

  • Understand the scope of notes to be removed from translation files
  • Create a script/method to remove outdated file source and file path notes
  • Remove notes with category="file-source" from all translation files
  • Remove notes with file paths (containing Part-DB1\ or templates\ or src\) from all translation files
  • Keep notes with category="state" as they are not file path related
  • Keep notes without category that contain state values (like "new")
  • Preserve original XML declaration format with double quotes
  • Verify the changes don't break the translation system (all XML files validated)
  • Request code review (passed with no issues)

Summary

Successfully removed 20,736 outdated notes from 65 translation files, deleting 33,623 lines of outdated file source and path references. All state notes (3,153 total) were preserved, XML declaration format preserved with double quotes, and all XML files remain valid.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jbtronics <5410681+jbtronics@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove outdated notes from translations Remove outdated file source and path notes from translation files Feb 7, 2026
Copilot AI requested a review from jbtronics February 7, 2026 16:42
@jbtronics
Copy link
Member

Do not replace the double quotes with single quotes in tag

@jbtronics
Copy link
Member

@copilot Do not replace the double quotes with single quotes in tag

Co-authored-by: jbtronics <5410681+jbtronics@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 7, 2026

@copilot Do not replace the double quotes with single quotes in tag

Fixed in commit b7778b6. The XML declaration now preserves the original double quotes format.

@jbtronics jbtronics marked this pull request as ready for review February 7, 2026 17:33
@jbtronics jbtronics merged commit f5841cc into master Feb 7, 2026
19 checks passed
@jbtronics jbtronics deleted the copilot/remove-outdated-notes-from-translations branch February 7, 2026 17:33
@codecov
Copy link

codecov bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.71%. Comparing base (dcdc990) to head (b7778b6).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1225      +/-   ##
============================================
- Coverage     54.72%   54.71%   -0.01%     
  Complexity     7918     7918              
============================================
  Files           596      596              
  Lines         25468    25468              
============================================
- Hits          13937    13935       -2     
- Misses        11531    11533       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants