Need QC for %s
@dosumis is already working on this.
Since Sep 4, 2020.
Assessment
This issue has not been assessed yet.
Description
One of the most important pain points during validation is cases when I am using % instead of %s in pattern strings strings. Example:
name:
text: "% adenoma"
vars:
- v0
def:
text: "A adenoma that involves the %."
vars:
- v0
This should be carefully QCed, by taking also into account the number of variables in the subsequent vars section; This could avoid mistakenly flagging up cases like like
def:
text: "A adenoma that involves at least 90% the %s."
vars:
- v0
or
def:
text: "A adenoma that involves at least 90 % the %s."
vars:
- v0
- Dominant language
- Python
- Stars
- 49
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from INCATools/dead_simple_owl_design_patterns
-
`test_text_fields` validator ignores `dataProperties` and `annotationProperties`Possibly taken @forest-savage1234 claimed this 55 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Request: new field for synonym permutationsPossibly taken @matentzn claimed this 319 days ago. Open
Difficulty 5/5 Over a week Newbie friendliness 30/100
INCATools/dead_simple_owl_design_patterns#87 · 2 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
schema_ext
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
INCATools/dead_simple_owl_design_patterns#67 · 3 comments ·
All issues in INCATools/dead_simple_owl_design_patterns
Similar issues
-
The changelog workflow excludes otelbot but not dependabot, so every Dependabot pull request fails the changelog checkPossibly taken @ocelotl claimed this today. Open
Difficulty 1/5 Under an hour Newbie friendliness 82/100
open-telemetry/opentelemetry-python#5825 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
cuga-project/cuga-agent#1020 ·
Maintainers usually reply within 2 days
-
Difficulty 1/5 Under an hour Newbie friendliness 70/100
microsoft/playwright#43640 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
stac-utils/stac-fastapi-elasticsearch-opensearch#965 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day