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

Use N3.js instead of regex for parsing .nt files. #174

Merged
merged 1 commit into from
Feb 24, 2022
Merged

Conversation

Eyas
Copy link
Collaborator

@Eyas Eyas commented Feb 24, 2022

This technically also allows us to use any format other than .nt, but I have not verified that this works.

This is a further milestone in supporting #169

Still need to actually understand blank nodes, etc.

In the meantime, however, using N3 allows us to:

  • Recognize blank strings
  • Properly display unicode in comments
  • Make sure characters are always unescaped in comments

This technically also allows us to use any format other than .nt, but I have not verified that this works.
@Eyas Eyas merged commit 9bafa62 into google:main Feb 24, 2022
@Eyas Eyas deleted the usen3 branch February 24, 2022 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant