Skip to content

Add crash stats output to git ignore#19369

Merged
SaschaCowley merged 1 commit into
betafrom
gitIfnoreCrashStats
Dec 12, 2025
Merged

Add crash stats output to git ignore#19369
SaschaCowley merged 1 commit into
betafrom
gitIfnoreCrashStats

Conversation

@SaschaCowley

Copy link
Copy Markdown
Member

Link to issue number:

Follow-up #19175

Summary of the issue:

If NVDA crashes when running from source, git picks up nvda_crash_stats.txt as an untracked file.

Description of user facing changes:

None

Description of developer facing changes:

Git will ignore nvda_crash_stats.txt, meaning it's much less likely to be accidentally committed and doesn't clutter our git output.

Description of development approach:

Added source/nvda_crash_stats.txt to .gitignore.

Testing strategy:

Ran git status. Verified that nvda_crash_stats was no longer showing as an untracked file.

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.

@SaschaCowley SaschaCowley marked this pull request as ready for review December 11, 2025 06:41
@SaschaCowley SaschaCowley requested a review from a team as a code owner December 11, 2025 06:41
@seanbudd seanbudd added this to the 2026.1 milestone Dec 11, 2025
@SaschaCowley SaschaCowley merged commit 5093ac0 into beta Dec 12, 2025
39 of 41 checks passed
@SaschaCowley SaschaCowley deleted the gitIfnoreCrashStats branch December 12, 2025 01:17
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