Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

[Python] Sphinx doesn't recognize closing backticks when not adjacent to word boundary #401

Closed
geigerj opened this issue Aug 19, 2016 · 8 comments
Assignees
Labels
lang: python Issues specific to Python. priority: p2 Moderately-important priority. Fix may not be included in next release. 🚨 This issue needs some love. status: won't fix type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@geigerj
Copy link
Contributor

geigerj commented Aug 19, 2016

Observed behavior:

Build vision docs:

python execute_pipeline.py \
    --config "../googleapis/gapic/api/artman_vision.yaml,\
    ../googleapis/gapic/lang/doc.yaml" \
    --language python \
    GapicClientPipeline 
cd output
tox -e docs

fails with

/usr/local/google/home/jgeiger/repos/gapi-dev/artman/output/gcloud-python-cloud-vision/.tox/docs/local/lib/python2.7/site-packages/google/cloud/vision/v1/image_annotator_pb2.py:docstring of google.cloud.vision.v1.image_annotator_pb2.EntityAnnotation:38: WARNING: Inline literal start-string without end-string.

Cause

Appears to be that generated docstring for EntityAnnotator contains the text

``boundingPoly``s

Sphinx does not recognize the second pair of backticks because it is not at a word boundary

Solution

TBD

/cc @bjwatson

@geigerj geigerj added lang: python Issues specific to Python. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Aug 19, 2016
@geigerj
Copy link
Contributor Author

geigerj commented Dec 21, 2016

@lukesneeringer

@geigerj
Copy link
Contributor Author

geigerj commented Dec 21, 2016

@evaogbe evaogbe added the type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. label Aug 14, 2017
@evaogbe
Copy link
Contributor

evaogbe commented Nov 29, 2017

@lukesneeringer @geigerj Is this still relevant?

@geigerj
Copy link
Contributor Author

geigerj commented Nov 29, 2017

I'd assume so, but @lukesneeringer to confirm.

@andreamlin
Copy link
Contributor

This is possible related to #401

@vchudnov-g
Copy link
Contributor

@lukesneeringer Any updates (to meet our SLO)?

@JustinBeckwith JustinBeckwith added the 🚨 This issue needs some love. label Jun 27, 2019
@vchudnov-g
Copy link
Contributor

@lukesneeringer @hzyi-google Any updates on this? We're out of SLO.

@software-dov
Copy link
Contributor

Leaning towards won't fix due to microgenerator transition timeline

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lang: python Issues specific to Python. priority: p2 Moderately-important priority. Fix may not be included in next release. 🚨 This issue needs some love. status: won't fix type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

7 participants