-
Notifications
You must be signed in to change notification settings - Fork 13
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
3.x changes and more #74
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add `inputFormat` option. - Use "application/n-quads" for a N-Quads string that will be parsed. - Omit option for a JSON dataset or legacy dataset. - This can simplify a common case of using the internal parser to generate a dataset.
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
The `inputFormat` change can break callers that mistakenly pass in an incorrect `inputFormat`. jsonld.js was doing this. Backing this out for now and it will be added to a future major version release.
- Add testing on Node.js 20.x. - Run tools on 20.x.
This reverts commit 0c3118a.
- Move restored `inputFormat` text to the latest version.
- Fixes Node.js 20.x issues.
davidlehn
force-pushed
the
main-with-3.4.0
branch
from
June 22, 2023 22:44
e7d78c0
to
453dad7
Compare
- Add eslint jsdoc support. - Fix lint issues.
**BREAKING**: Change "URDNA2015" to "RDFC-1.0" to match latest spec changes. Use of "URDNA2015" as a named algorithm option is now deprecated and will cause a warning.
- Add docs. - Add Algorithm Support docs. - Add URDNA2015 Migration docs. - Add migration tools: - Undocumented "rejectURDNA2015" API option to not allow URDNA2015. - Allows concerned code to explicitly fail in potential edge cases. - Global "RDF_CANONIZE_TRACE_URDNA2015" to trace URDNA2015 use. - Allows developers to find deep usage of URDNA2015 more easily. - These may exist as long as the URDNA2015 alias support exists. Both are not expected to have security implications.
**BREAKING**: Check output `format` parameter. Must be omitted, falsey, or "application/n-quads".
- Remove advanced "useNative" example. - Add common N-Quads input example.
- Add `maxWorkFactor` to calculate `maxDeepIterations` based on non-unique blank nodes. Default to `1`. - If `maxDeepIterations` is >= 0 then use it explicitly. - Remove default timeout signal. - Add docs. - Add tests. - Update async and sync versions. - Update tests to handle `computationalComplexity` test paramter and map it to `maxWorkFactor` adjustments. - Improve negative test handling.
Add support for generalized RDF `BlankNode` predicate during N-Quads serialization.
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
- Fix maxWorkFactor === 0 use case. - Improve error if N-Quads input not a string. - Fix calls without options. - Test error messages to ensure correct errors are thrown. - Remove default options not needed for testing. - Add various tests for complexity controls, edge cases, and coverage. - Add simple duplicate quads test.
- Rename `URDNA2015.js` to `RDFC10.js`. - Rename 'URDNA2015` class to `RDFC10`. - Rename `URDNA2015Sync.js` to `RDFC10Sync.js`. - Rename 'URDNA2015Sync` class to `RDFC10Sync`.
- Add package.json section. - Add instructions.
- Add simple invalid N-Quads test. - Start of IRI escaping test.
When set to truthy string will setup tests that are run for official test result reports.
davidlehn
force-pushed
the
main-with-3.4.0
branch
from
October 5, 2023 03:13
4884d85
to
a01c5ae
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
canonicalIdMap
feature. Cherry-pick was used instead of a merge commit by request. Future 3.x changes will be cherry picked as needed.