search

nvie / vim-rst-tables

Easily create and reformat your RST (reStructuredText) tables as you change cell content.
121 stars 36 forks source link

MRG: refactor for Python 3, testing rst_tables.vim #30

Closed matthew-brett closed 5 years ago

matthew-brett commented 5 years ago

Looking at the recent commits, I think y'all are not using the split vim / Python files in src, so this PR removes these copies, and sets the tests to run from the rst_tables.vim code.

Also, allow execution on Python 3.

  1. refactor to use Python 3 idioms for map;
  2. refactor to do decoding / encoding once (and not for Py3);
  3. remove src directory and mention of build.py in README.
  4. refactor testing to read Python code from rst_tables.vim
  5. fix tests to current APIs.

The tests now pass for me on Pythons 3 and 2.

matthew-brett commented 5 years ago

I should say - I needed my proposed patches to vim_bridge to get the tests to work - see : https://github.com/nvie/vim_bridge/pull/9

nvie commented 5 years ago

This is great, @matthew-brett — thanks! 🙏