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.

Sign up for GitHub

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

Jump to bottom

Add missing useMemo import #2962

Merged

koesie10 merged 1 commit into main from koesie10/use-memo

Oct 12, 2023

Member

koesie10 commented Oct 12, 2023

It seems like there was a semantic merge conflict between #2959 and #2952 which breaks the build. This adds the missing useMemo import.

Checklist

CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
Issues have been created for any UI or other user-facing changes made by this pull request.
[Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.


          Add missing useMemo import

ea8e5c6

koesie10 requested a review from a team as a code owner

October 12, 2023 11:14

charisk approved these changes

View reviewed changes

koesie10 enabled auto-merge

October 12, 2023 11:19

kaspersv mentioned this pull request

Hash result set name in sorted result set path #2955

Merged

3 tasks

koesie10 merged commit 00a5717 into main

14 checks passed

koesie10 deleted the koesie10/use-memo branch

October 12, 2023 11:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet