Skip to content

Minor refactoring & code/comment cleanup#1494

Merged
tombogle merged 3 commits intomasterfrom
minor-code-cleanup
Mar 26, 2026
Merged

Minor refactoring & code/comment cleanup#1494
tombogle merged 3 commits intomasterfrom
minor-code-cleanup

Conversation

@tombogle
Copy link
Copy Markdown
Contributor

@tombogle tombogle commented Mar 19, 2026


Open with Devin

This change is Reviewable

@tombogle tombogle self-assigned this Mar 19, 2026
@tombogle tombogle changed the title Minor refactoring & code/comment cleanup. Minor refactoring & code/comment cleanup Mar 19, 2026
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 19, 2026

Palaso Tests

     3 files   -     1       3 suites   - 1   7m 23s ⏱️ - 3m 1s
 5 095 tests ±    0   4 861 ✅  -     1  234 💤 +  1  0 ❌ ±0 
11 483 runs   - 5 114  11 008 ✅  - 4 869  475 💤  - 245  0 ❌ ±0 

Results for commit 2e6a66e. ± Comparison against base commit ad0b279.

This pull request skips 1 test.
SIL.Tests.IO.FileLocationUtilitiesTests ‑ LocateInProgramFiles_SendValidProgramDeepSearch_ReturnsProgramPath

♻️ This comment has been updated with latest results.

@tombogle tombogle removed the request for review from imnasnainaec March 26, 2026 21:19
Copy link
Copy Markdown
Contributor

@andrew-polk andrew-polk left a comment

Choose a reason for hiding this comment

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

@andrew-polk reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on tombogle).

@tombogle tombogle enabled auto-merge (squash) March 26, 2026 21:30
@tombogle tombogle merged commit ce83cef into master Mar 26, 2026
16 of 17 checks passed
@tombogle tombogle deleted the minor-code-cleanup branch March 26, 2026 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants