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

Spurious warning with enum symbol names containing colon #645

Open
tetron opened this issue Jan 10, 2023 · 0 comments
Open

Spurious warning with enum symbol names containing colon #645

tetron opened this issue Jan 10, 2023 · 0 comments

Comments

@tetron
Copy link
Member

tetron commented Jan 10, 2023

2023-01-05T23:13:49.598203507Z URI prefix '#bcftools_view.cwl/bcftools-view/min_count/min_count/min_count_type/min_count_type/' of '#bcftools_view.cwl/bcftools-view/min_count/min_count/min_count_type/min_count_type/:nref' not recognized, are you missing a $namespaces section?

It is getting confused by the colon in the enum symbol name ":nref". The code is interpreting it as a possible prefix (like "sbg:") when in fact it is just part of the name.

I think the check could be a little more nuanced and does not need to print a warning in this case, because the rest of the code actually handles it just fine.

These sorts of warnings should also provide a line number.

@tetron tetron changed the title Spurious warning with symbol names containing colon Spurious warning with enum symbol names containing colon Jan 10, 2023
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