-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
adds json-ld serialization #80
base: master
Are you sure you want to change the base?
adds json-ld serialization #80
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #80 +/- ##
=============================================
- Coverage 97.28% 79.20% -18.08%
- Complexity 67 80 +13
=============================================
Files 6 7 +1
Lines 184 226 +42
=============================================
Hits 179 179
- Misses 5 47 +42 ☔ View full report in Codecov by Sentry. |
@kghbln @JeroenDeDauw @gesinn-it-gea can we merge this? |
There is no automated test. Did someone manually test this? |
Some docs on expected behavior could also help. In the release notes or other documentation pages |
ok, I will add a test, it won't take long |
I am using it on some wikis. |
I see two deprecation notices in MW 1.39:
@thomas-topway-it can you have a look, please? |
adds json-ld serialization using https://github.com/easyrdf/easyrdf
from SMW's standard RDF serialization
for testing purpose run
composer install
in the extension folder after merging the pull request #79