Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixed image in README.rst #55

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

Merged
merged 4 commits into from
May 9, 2019
Merged

Fixed image in README.rst #55

merged 4 commits into from
May 9, 2019

Conversation

evaherrada
Copy link
Collaborator

No description provided.

@evaherrada
Copy link
Collaborator Author

Anyone have any clue why Travis is not passing? I've verified that the image is readable.

@sommersoft
Copy link
Collaborator

@dherrada looks like adding ../ before docs succeeds:

~/Dev/circuitpython_libs/Adafruit_CircuitPython_CircuitPlayground/docs:$> git diff
diff --git a/README.rst b/README.rst
index 3b69922..73c3f7e 100644
--- a/README.rst
+++ b/README.rst
@@ -16,7 +16,7 @@ Introduction
 
 This high level library provides objects that represent CircuitPlayground hardware.
 
-.. image :: /_static/circuitplayground_express.jpg
+.. image :: ../docs/_static/circuitplayground_express.jpg
     :target: https://adafruit.com/product/3333
     :alt: CircuitPlayground Express

~/Dev/circuitpython_libs/Adafruit_CircuitPython_CircuitPlayground/docs:$> sphinx-build -E -W -b html . _build/html
Running Sphinx v2.0.0
making output directory... done
loading intersphinx inventory from https://docs.python.org/3.4/objects.inv...
loading intersphinx inventory from https://circuitpython.readthedocs.io/en/latest/objects.inv...
loading intersphinx inventory from https://circuitpython.readthedocs.io/projects/neopixel/en/latest/objects.inv...
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 3 source files that are out of date
updating environment: 3 added, 0 changed, 0 removed
reading sources... [100%] index                                                                         
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] index                                                                          
generating indices... genindex py-modindex
highlighting module code... [100%] adafruit_circuitplayground.express                                   
writing additional pages... search/home/sommersoft/.local/lib/python3.6/site-packages/sphinx_rtd_theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {{ super() }}

copying images... [100%] ../docs/_static/circuitplayground_express.jpg                                  
copying static files... done
copying extra files... done
dumping search index in English (code: en) ... done
dumping object inventory... done
build succeeded.

The HTML pages are in _build/html.

@evaherrada
Copy link
Collaborator Author

Thanks for the help!

Copy link
Collaborator

@sommersoft sommersoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the image fix!

@sommersoft sommersoft merged commit 473d554 into adafruit:master May 9, 2019
@evaherrada
Copy link
Collaborator Author

@sommersoft So it looks like the only way to get travis to pass is to use ../docs, and the only way to have the image show up is with docs or ./docs

adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request May 25, 2019
Updating https://github.com/adafruit/Adafruit_CircuitPython_CircuitPlayground to 2.0.6 from 2.0.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_CircuitPlayground#58 from dherrada/master
  > Merge pull request adafruit/Adafruit_CircuitPython_CircuitPlayground#55 from dherrada/master
  > Merge pull request adafruit/Adafruit_CircuitPython_CircuitPlayground#53 from karengineering/patch-1
  > Merge pull request adafruit/Adafruit_CircuitPython_CircuitPlayground#48 from adafruit/revert-47-add-blinka-to-requirements-txt
  > Merge pull request adafruit/Adafruit_CircuitPython_CircuitPlayground#47 from demophoon/add-blinka-to-requirements-txt

Updating https://github.com/adafruit/Adafruit_CircuitPython_FocalTouch to 1.1.4 from 1.1.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_FocalTouch#8 from dherrada/master
  > Merge pull request adafruit/Adafruit_CircuitPython_FocalTouch#7 from TG-Techie/patch-2

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Added the following libraries: Adafruit_CircuitPython_AzureIoT
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

Successfully merging this pull request may close these issues.

2 participants