Skip to content

change(web): prep for enhanced flick-resets - alters the contact-path evaluation signature 🐵#10152

Merged
jahorton merged 2 commits intofeature-gesturesfrom
change/web/path-evaluate-signature
Dec 13, 2023
Merged

change(web): prep for enhanced flick-resets - alters the contact-path evaluation signature 🐵#10152
jahorton merged 2 commits intofeature-gesturesfrom
change/web/path-evaluate-signature

Conversation

@jahorton
Copy link
Copy Markdown
Contributor

@jahorton jahorton commented Dec 6, 2023

@github-actions github-actions bot added the web/ label Dec 6, 2023
@keymanapp-test-bot keymanapp-test-bot bot changed the title change(web): prep for enhanced flick-resets - alters the contact-path evaluation signature change(web): prep for enhanced flick-resets - alters the contact-path evaluation signature 🐵 Dec 6, 2023
@keymanapp-test-bot keymanapp-test-bot bot added this to the A17S27 milestone Dec 6, 2023
itemPriority: 1,
pathModel: {
evaluate: (path, baseStats) => {
evaluate: (path, dummy, dummy2, baseStats) => {
Copy link
Copy Markdown
Contributor Author

@jahorton jahorton Dec 6, 2023

Choose a reason for hiding this comment

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

This evaluate function as a whole will be changing in a near-followup, so I was lazy with the dummy-var naming here.

@jahorton jahorton force-pushed the change/web/path-evaluate-signature branch from 4fa268d to ef247ba Compare December 8, 2023 04:53
@jahorton jahorton changed the base branch from feature-gestures to feat/web/gesture-path-memory-optimization December 8, 2023 04:53
@mcdurdin mcdurdin modified the milestones: A17S27, A17S28 Dec 8, 2023
@jahorton jahorton marked this pull request as ready for review December 9, 2023 02:32
Base automatically changed from feat/web/gesture-path-memory-optimization to feature-gestures December 12, 2023 00:31
@jahorton jahorton merged commit 207324e into feature-gestures Dec 13, 2023
@jahorton jahorton deleted the change/web/path-evaluate-signature branch December 13, 2023 07:09
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.

3 participants