Skip to content

Conversation

@darcywong00
Copy link
Contributor

@darcywong00 darcywong00 commented Aug 14, 2025

End of sprint task

Test-bot: skip

mcdurdin and others added 20 commits August 5, 2025 14:31
Test bot has also just been installed, so this serves as a test bot
verification also.

Relates-to: keymanapp/keyman#559
Test-bot: skip
Test bot has also just been installed, so this serves as a test bot
verification also.

Relates-to: keymanapp/keyman#559
Test-bot: skip
Co-authored-by: Darcy Wong <darcy_wong@sil.org>
We started to see situations where status checks for new PRs would fail,
for example see #290, with hard to parse errors about actual search
results mismatching expected results. After digging in, with a lot of
logging and debugging, I found that the issue was a race where the test
would run before full text indexing had completed. This change delays
the completion of the database build until the full text indexing has
finished.

The full description of my investigation is recorded in #290 for future
reference.
Keeping in git log for posterity
fix: wait for full text indexing to complete when building database
maint: add labeler for issues and PRs
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Aug 14, 2025

User Test Results

Test specification and instructions

User tests are not required

@darcywong00 darcywong00 merged commit 70027fa into staging Aug 15, 2025
5 checks passed
@darcywong00 darcywong00 deleted the chore/merge-master-to-staging-a19s9 branch August 15, 2025 06:06
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants