Skip to content

Conversation

Esteban82
Copy link
Member

Add requirements to add youtube links in the rst files.

Is it ok @seisman

Closes #36.

@Esteban82 Esteban82 added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 23, 2024
@Esteban82 Esteban82 requested a review from seisman October 23, 2024 11:26
@Esteban82 Esteban82 self-assigned this Oct 23, 2024
@seisman
Copy link
Member

seisman commented Oct 23, 2024

It should be added to environment.yml, too.

@Esteban82
Copy link
Member Author

Esteban82 commented Oct 23, 2024

Ok. What should I add there?

  1. This?
sphinx.ext.intersphinx
sphinxcontrib.youtube
  1. Or this?

sphinxcontrib-youtube==1.4.1

@seisman
Copy link
Member

seisman commented Oct 23, 2024

Just add sphinxcontrib-youtube.

@Esteban82 Esteban82 merged commit eb75ecc into main Oct 23, 2024
3 checks passed
@Esteban82 Esteban82 deleted the Add_video_support branch October 23, 2024 12:02
weiji14 added a commit to GenericMappingTools/agu24workshop that referenced this pull request Dec 1, 2024
weiji14 added a commit to GenericMappingTools/agu24workshop that referenced this pull request Dec 2, 2024
* Add docs for tutorial 6

* Use sphinx.ext.extlinks for gmt-module links

Xref https://www.sphinx-doc.org/en/master/usage/extensions/extlinks.html

* Add sphinxcontrib-youtube

Xref GenericMappingTools/gmt-examples#37

* Add sphinx_gmt.gmtplot

Sphinx extension used to define the `gmtplot` directive that will execute the given GMT code and insert the generated figure into the document. Need to set `highlight_language: bash` otherwise there will be an `UnboundLocalError: cannot access local variable 'suffix' where it is not associated with a value`.

---------

Co-authored-by: Wei Ji <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to add videos to the examples?

2 participants