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

Static local contexts need to be kept in sync. #112

Open
davidlehn opened this issue Dec 18, 2021 · 1 comment
Open

Static local contexts need to be kept in sync. #112

davidlehn opened this issue Dec 18, 2021 · 1 comment

Comments

@davidlehn
Copy link
Member

The static contexts in this library need to be kept in sync with upstream contexts. How should this be accomplished?

Contexts:

One approach would be to run a github action check, probably on PRs and periodically (weekly?). In a recent PR I used diff, curl, and jq to compare what this module serves and the live dereferenced resources:
#111

I don't know what would happen if this reported diffs that were not desired in this package. It would be good to at least know.

@davidlehn
Copy link
Member Author

Two other options to at least help with comparisons. Instead of inline data in JS, inline the JSON direct:

module.exports =
/* BEGIN CONTEXT */
{
  "@context": ...
  "foo": ...
}
/* END CONTEXT */
;

Or put the contexts in webpack/node loader friendly .json files and require() them. I think that would work with the current structure?

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

No branches or pull requests

1 participant