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

strange null Warnings #112

Closed
JJ-Author opened this issue Mar 1, 2020 · 5 comments
Closed

strange null Warnings #112

JJ-Author opened this issue Mar 1, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@JJ-Author
Copy link

seems not to work an returns strange warnings
https://github.com/dbpedia/format-mappings/runs/477652228?check_suite_focus=true

Run rdflint3s
##[error]Process completed with exit code 1.
Run rdflint
tarql/1.ttl
  ERROR  RDF parse warning: [line: 1, col: 1] null

tarql/2.ttl
  ERROR  RDF parse warning: [line: 1, col: 1] null

tarql/3.ttl
  ERROR  RDF parse warning: [line: 1, col: 1] null

##[error]Process completed with exit code 1.
@foooomio
Copy link
Member

foooomio commented Mar 1, 2020

Hi, thanks for reporting.

This problem seems to be with rdflint itself, not setup-rdflint. Run rdflint locally and you'll see the same errors.

For now transfering the issue to imas/rdflint.

@foooomio foooomio transferred this issue from imas/setup-rdflint Mar 1, 2020
@foooomio foooomio closed this as completed Mar 1, 2020
@foooomio foooomio reopened this Mar 1, 2020
@foooomio
Copy link
Member

foooomio commented Mar 1, 2020

Based on my research, the warnings appear when neither the command line option baseuri nor the config option baseUri is specified.

Is it intentional? @takemikami

@JJ-Author
Copy link
Author

JJ-Author commented Mar 1, 2020

Thanks for the quick reply. I don't fully understand this. When reading from a file usually the base URI is set to the filename / filepath. It could additionally make sense to have a config file in the action which works out of the box. For validation / linting the baseUri is not important. I fail to understand the config file since the docu is in Japanese and Google refuses to translate it ^^. So yes the issue might be in the right place now, but I think a working default config in the action setup makes sense, doesn't it?

@takemikami takemikami added the bug Something isn't working label Mar 1, 2020
@takemikami
Copy link
Member

takemikami commented Mar 1, 2020

This strange warnings is a bug, and I will fix correctly in default configuration.

If you need to use rdflint shortly,
please put configuration file like following, and you can avoid this bug.

rdflint-config.yml

baseUri: http://example.com/

@takemikami
Copy link
Member

fixed, at v0.1.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants