-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Index fixes #4322
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
Index fixes #4322
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
b7d29d0
to
9bfa737
Compare
6c3756a
to
7c7fa03
Compare
7c7fa03
to
c5e6290
Compare
Hey @JubilantJerry , apologies for the slow review time here, it's been a busy past few weeks prepping our 1.0 launch. This all looks solid but mind sharing a bit of an overview of what these changes entail? Also, if there's any tests you could write in |
@JubilantJerry I went through some of the issues you linked actually and that provides plenty of context for the changes here. I pushed up a quick merge conflict resolution just now, and once the tests pass here I'm going to merge to main. However, if there's any test you think would be valuable to add to help us prevent future regressions that would be a great followup. Thanks again for all the huge improvements here, and for documenting the issues in the first place! |
Description
Fixes for indexing and
@codebase
issues #4239, #4252, #4254, #4255