Skip to content

Multiple Langauge support#196

Closed
chsasank wants to merge 8 commits intosphinx-gallery:masterfrom
chsasank:polyglot
Closed

Multiple Langauge support#196
chsasank wants to merge 8 commits intosphinx-gallery:masterfrom
chsasank:polyglot

Conversation

@chsasank
Copy link
Copy Markdown

@chsasank chsasank commented Jan 16, 2017

Hi @Titan-C,

Following discussion #194, I've made a working version with support for multiple languages. In the process, I've done quite a bit of refactoring.

I also

  • made some functions 'private' with prepending _ to function names.
  • made some code pep8 complaint
  • Added comments here and there

What I've done was I've forked func_rst branch of your fork from #160 and added my commits. Unfortunately, you've made the top few commits in your branch obsolete since my fork, probably using commit --amend / squash and push --force. There is a bug (:overflow: being visible in gallery page) which you've fixed, but I couldn't merge these changes easily because of the above :(

For further commits you add to #160, please add your commits one by one rather than modifying existing commits. We can rebase/package these commits into 1 after PR is ready to merge.

Please have a look at the implementation and give me your feedback.

Right now, I've written only lua parser, adding other languages shouldn't be hard at this point.

TODO:

  • Merge changes from Titan-C/sphinx-gallery:func_rst carefully.
  • Add other language support
  • Write Tests

Titan-C and others added 8 commits December 23, 2016 14:35
This allows Sphinx-Gallery to be build by a direct call to
sphinx-build, specifying the sources directory and the build output
directory. There is no more the constrain to launch the build from the
makefile is the sources directory.

- Correct relative path when embedding links
- Correct path during build
- Summary of failing examples to work on relative path builds
- Update tests
- Update docs clean and build configs
New modules
- execute blocks
- save images
- write rst

review image test

Load Agg in execute_blocks.py because tests load order is alphabetic
@lesteve
Copy link
Copy Markdown
Member

lesteve commented Jan 17, 2017

Wow kudos for the effort ! I thought I commented on the issue but apparently not, I'll do that in a few minutes.

@chsasank chsasank mentioned this pull request Jan 17, 2017
4 tasks
@chsasank
Copy link
Copy Markdown
Author

Closing and opening another branch #197 with fixed commit history.

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.

3 participants