Skip to content

Typography corrections#317

Open
codam wants to merge 1 commit into
linkeddata:mainfrom
codam:patch-1
Open

Typography corrections#317
codam wants to merge 1 commit into
linkeddata:mainfrom
codam:patch-1

Conversation

@codam

@codam codam commented Jun 1, 2019

Copy link
Copy Markdown
Contributor

Thanks for the good work!

@megoth megoth left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

jeswr added a commit that referenced this pull request Jul 5, 2026
* chore: add prepare script so git-dependency installs get built (#654)

Installing rdflib from a git URL yielded a package whose main entry
lib/index.js does not exist, because lib/ is gitignored and nothing
built it. 'prepare' is npm's documented hook for building git
dependencies; it runs on 'npm install' from git (and in local dev
installs), so consumers get a working lib/.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* chore: remove unused jsonparser (#87)

src/jsonparser.js is not used by any code path: parse() handles
application/ld+json via src/jsonldparser.js, and the only reference
to jsonParser anywhere in the repo was the re-export in src/index.ts.
It is undocumented, untested, and expects a bespoke non-JSON-LD input
shape, so remove it (this drops the vestigial $rdf.jsonParser export).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix: replace non-existent kb.load() in example/people/book.js (#88)

The store has never had a load() method in modern rdflib; loading is
done by the Fetcher. Use fetcher.load() for the friends' profile
documents and rdfs:seeAlso links, and move the friends processing
inside the initial fetch callback so it runs once the profile has
actually been loaded (previously it ran synchronously against a
store that had not been populated yet).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: point README data-model compatibility link at the RDF/JS spec (#319)

The old link to rdfjs/representation-task-force interface-spec.md has
been a 404 for years; the spec now lives at rdf.js.org. Re-applies the
intent of PR #281 by @darrengarvey, updated to link directly to the
data model spec document.

Co-authored-by: Darren Garvey <dagarvey@ebay.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: document the literal() factory signature (#328)

The literal() factory takes (value, languageOrDatatype) per the RDF/JS
DataFactory interface; the three-argument literal(value, undefined,
datatype) form shown in older tutorials silently ignores the datatype
and yields a plain xsd:string literal. Document the correct usage and
the pitfall in the README.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: document owl:sameAs smushing and link#uri statements (#221)

Explain when the store adds <canonical> link:uri <obsoleted>
statements (only when smushing features are enabled, off by default
since #458), what they are for (alias book-keeping behind
store.uris()/allAliases()), and why they can show up when querying
or serializing a smushing store.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: re-apply typo and code-sample fixes to webapp-intro from PR #318

Re-applies the still-relevant parts of stale PR #318 by @codam
(2019): typo fixes, corrected code samples (cardFor(), fetcher
spacing, the update() insert-parameter description) and the broken
block-diagram image path. Hunks already fixed on main in the
meantime were skipped.

Co-authored-by: Raphaël Harmel <raphael.harmel@gmail.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: document the SPARQL subset supported by SPARQLToQuery (#301)

Add a 'Querying the store with SPARQL' section to the webapp intro:
how to run a query with SPARQLToQuery + store.query()/querySync(),
how bindings are delivered, the auto-dereferencing behavior, and an
honest list of what the parser does and does not support (SELECT
only, PREFIX, basic graph patterns, OPTIONAL, a minimal FILTER
language; no UNION/DESCRIBE/LIMIT/etc).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* docs: re-apply typo corrections to turtle-intro from PR #317

Re-applies stale PR #317 by @codam (2019), which fixes a batch of
spelling and grammar errors in Documentation/turtle-intro.html. The
patch still applied cleanly.

Co-authored-by: Raphael Harmel <raphael.harmel@gmail.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Darren Garvey <dagarvey@ebay.com>
Co-authored-by: Raphaël Harmel <raphael.harmel@gmail.com>
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.

2 participants