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

fix: indexing quoted triples #369

Merged
merged 5 commits into from
Oct 27, 2023
Merged

fix: indexing quoted triples #369

merged 5 commits into from
Oct 27, 2023

Conversation

jeswr
Copy link
Collaborator

@jeswr jeswr commented Oct 20, 2023

This work was originally in #341, but have pulled into a separate PR as this can be put in a patch release.

There are no performance degredations against N3Store-perf.js

@jeswr jeswr marked this pull request as draft October 20, 2023 15:05
@jeswr jeswr marked this pull request as ready for review October 20, 2023 15:13
@jeswr jeswr changed the title fix/indexing quoted triples fix: indexing quoted triples Oct 20, 2023
test/N3Store-test.js Outdated Show resolved Hide resolved
test/N3Store-test.js Outdated Show resolved Hide resolved
Copy link
Member

@RubenVerborgh RubenVerborgh left a comment

Choose a reason for hiding this comment

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

Okay that's quite cool 😎 Looks good to me!

@jeswr jeswr merged commit e1cc8dd into main Oct 27, 2023
11 checks passed
@jeswr jeswr deleted the fix/indexing-quoted-triples branch October 27, 2023 13:58
@github-actions
Copy link

🎉 This PR is included in version 1.17.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants