Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(web): move lm-worker β†’ worker-thread πŸ—οΈ #12150

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

ermshiperete
Copy link
Contributor

Move common/web/lm-worker/ β†’ web/src/engine/predictive-text/worker-thread/.

Fixes: #12148

@keymanapp-test-bot skip

@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Aug 9, 2024

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot bot added this to the A18S8 milestone Aug 9, 2024
@ermshiperete ermshiperete force-pushed the refactor/web/12147_types branch from e7a3718 to 8fbec5f Compare August 12, 2024 17:45
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from 3ccd305 to 864a3bc Compare August 12, 2024 17:46
@ermshiperete ermshiperete force-pushed the refactor/web/12147_types branch from 8fbec5f to 6a6b7c4 Compare August 13, 2024 17:04
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from 864a3bc to e8572e3 Compare August 13, 2024 17:05
@ermshiperete ermshiperete force-pushed the refactor/web/12147_types branch from 6a6b7c4 to 6000f6d Compare August 14, 2024 17:32
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from e8572e3 to 28c2977 Compare August 14, 2024 17:32
@ermshiperete ermshiperete force-pushed the refactor/web/12147_types branch from 6000f6d to 392a0fb Compare August 15, 2024 10:27
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from 28c2977 to 08e0d30 Compare August 15, 2024 10:28
@ermshiperete ermshiperete force-pushed the refactor/web/12147_types branch from 392a0fb to 78857d9 Compare August 16, 2024 06:32
Move `common/web/lm-worker/` β†’
`web/src/engine/predictive-text/worker-thread/`.

Fixes: #12148
@ermshiperete ermshiperete force-pushed the refactor/web/12148_worker-thread branch from 08e0d30 to ea62ac2 Compare August 16, 2024 06:35
Base automatically changed from refactor/web/12147_types to master August 16, 2024 09:08
@ermshiperete ermshiperete marked this pull request as ready for review August 16, 2024 09:10
@darcywong00 darcywong00 modified the milestones: A18S8, A18S9 Aug 17, 2024
@jahorton
Copy link
Contributor

Something I noticed from using the latest changes from this PR's ancestors - please run an npm install command at the repository root after each stage; there may be subtle package-lock.json changes that result and that should be checked in.

(This goes for other move ____ -> ____ πŸ—οΈ PRs too.)

Addresses code review comments
@ermshiperete
Copy link
Contributor Author

Ahh, that's how you do it πŸ˜„

Done.

@ermshiperete ermshiperete merged commit 3c14a6e into master Aug 19, 2024
3 checks passed
@ermshiperete ermshiperete deleted the refactor/web/12148_worker-thread branch August 19, 2024 08:10
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 18.0.92-alpha

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.

refactor(web): common/web/lm-worker/ β†’ web/src/engine/predictive-text/worker-thread/
4 participants